diff options
-rw-r--r-- | components/script/dom/attr.rs | 31 | ||||
-rw-r--r-- | components/script/dom/bindings/trace.rs | 2 | ||||
-rw-r--r-- | components/script/dom/element.rs | 15 | ||||
-rw-r--r-- | components/script/dom/htmlfontelement.rs | 27 | ||||
-rw-r--r-- | components/script/dom/webidls/HTMLFontElement.webidl | 2 | ||||
-rw-r--r-- | components/style/properties.mako.rs | 21 | ||||
-rw-r--r-- | components/style/values.rs | 22 | ||||
-rw-r--r-- | components/util/str.rs | 106 | ||||
-rw-r--r-- | tests/wpt/metadata/html/dom/interfaces.html.ini | 6 | ||||
-rw-r--r-- | tests/wpt/metadata/html/dom/reflection-obsolete.html.ini | 129 | ||||
-rw-r--r-- | tests/wpt/mozilla/meta/MANIFEST.json | 6 | ||||
-rw-r--r-- | tests/wpt/mozilla/tests/mozilla/htmlfontelement_size_attribute.html | 34 |
12 files changed, 220 insertions, 181 deletions
diff --git a/components/script/dom/attr.rs b/components/script/dom/attr.rs index 70f315bcac3..2cd742ee422 100644 --- a/components/script/dom/attr.rs +++ b/components/script/dom/attr.rs @@ -18,6 +18,7 @@ use std::cell::Ref; use std::mem; use std::ops::Deref; use string_cache::{Atom, Namespace}; +use style::values::specified::Length; use util::str::{DOMString, parse_unsigned_integer, split_html_space_chars, str_join}; #[derive(JSTraceable, PartialEq, Clone, HeapSizeOf)] @@ -26,6 +27,7 @@ pub enum AttrValue { TokenList(DOMString, Vec<Atom>), UInt(DOMString, u32), Atom(Atom), + Length(DOMString, Option<Length>), } impl AttrValue { @@ -72,6 +74,11 @@ impl AttrValue { AttrValue::Atom(value) } + /// Assumes the `AttrValue` is a `TokenList` and returns its tokens + /// + /// ## Panics + /// + /// Panics if the `AttrValue` is not a `TokenList` pub fn as_tokens(&self) -> &[Atom] { match *self { AttrValue::TokenList(_, ref tokens) => tokens, @@ -79,6 +86,11 @@ impl AttrValue { } } + /// Assumes the `AttrValue` is an `Atom` and returns its value + /// + /// ## Panics + /// + /// Panics if the `AttrValue` is not an `Atom` pub fn as_atom(&self) -> &Atom { match *self { AttrValue::Atom(ref value) => value, @@ -86,9 +98,25 @@ impl AttrValue { } } + /// Assumes the `AttrValue` is a `Length` and returns its value + /// + /// ## Panics + /// + /// Panics if the `AttrValue` is not a `Length` + pub fn as_length(&self) -> Option<&Length> { + match *self { + AttrValue::Length(_, ref length) => length.as_ref(), + _ => panic!("Length not found"), + } + } + /// Return the AttrValue as its integer representation, if any. /// This corresponds to attribute values returned as `AttrValue::UInt(_)` /// by `VirtualMethods::parse_plain_attribute()`. + /// + /// ## Panics + /// + /// Panics if the `AttrValue` is not a `UInt` pub fn as_uint(&self) -> u32 { if let AttrValue::UInt(_, value) = *self { value @@ -105,7 +133,8 @@ impl Deref for AttrValue { match *self { AttrValue::String(ref value) | AttrValue::TokenList(ref value, _) | - AttrValue::UInt(ref value, _) => &value, + AttrValue::UInt(ref value, _) | + AttrValue::Length(ref value, _) => &value, AttrValue::Atom(ref value) => &value, } } diff --git a/components/script/dom/bindings/trace.rs b/components/script/dom/bindings/trace.rs index 94d3ad97c93..00a7f6c6c90 100644 --- a/components/script/dom/bindings/trace.rs +++ b/components/script/dom/bindings/trace.rs @@ -77,6 +77,7 @@ use std::sync::Arc; use std::sync::mpsc::{Receiver, Sender}; use string_cache::{Atom, Namespace}; use style::properties::PropertyDeclarationBlock; +use style::values::specified::Length; use url::Url; use util::str::{LengthOrPercentageOrAuto}; @@ -301,6 +302,7 @@ no_jsmanaged_fields!(WebGLError); no_jsmanaged_fields!(TimeProfilerChan); no_jsmanaged_fields!(MemProfilerChan); no_jsmanaged_fields!(PseudoElement); +no_jsmanaged_fields!(Length); impl JSTraceable for Box<ScriptChan + Send> { #[inline] diff --git a/components/script/dom/element.rs b/components/script/dom/element.rs index f99bd564dbe..d1db97128f0 100644 --- a/components/script/dom/element.rs +++ b/components/script/dom/element.rs @@ -71,7 +71,7 @@ use std::sync::Arc; use string_cache::{Atom, Namespace, QualName}; use style::legacy::{UnsignedIntegerAttribute, from_declaration}; use style::properties::DeclaredValue; -use style::properties::longhands::{self, background_image, border_spacing, font_family}; +use style::properties::longhands::{self, background_image, border_spacing, font_family, font_size}; use style::properties::{PropertyDeclaration, PropertyDeclarationBlock, parse_style_attribute}; use style::values::CSSFloat; use style::values::specified::{self, CSSColor, CSSRGBA}; @@ -302,6 +302,19 @@ impl RawLayoutElementHelpers for Element { font_family)]))))); } + let font_size = if let Some(this) = HTMLFontElementCast::to_ref(self) { + this.get_size() + } else { + None + }; + + if let Some(font_size) = font_size { + hints.push(from_declaration( + PropertyDeclaration::FontSize( + DeclaredValue::Value( + font_size::SpecifiedValue(font_size))))) + } + let cellspacing = if let Some(this) = HTMLTableElementCast::to_ref(self) { this.get_cellspacing() } else { diff --git a/components/script/dom/htmlfontelement.rs b/components/script/dom/htmlfontelement.rs index 030d1638c7e..873079bee11 100644 --- a/components/script/dom/htmlfontelement.rs +++ b/components/script/dom/htmlfontelement.rs @@ -7,17 +7,18 @@ use dom::attr::{Attr, AttrValue}; use dom::bindings::cell::DOMRefCell; use dom::bindings::codegen::Bindings::HTMLFontElementBinding; use dom::bindings::codegen::Bindings::HTMLFontElementBinding::HTMLFontElementMethods; -use dom::bindings::codegen::InheritTypes::{HTMLElementCast, HTMLFontElementDerived}; +use dom::bindings::codegen::InheritTypes::{ElementCast, HTMLElementCast, HTMLFontElementDerived}; use dom::bindings::js::Root; use dom::document::Document; -use dom::element::{AttributeMutation, ElementTypeId}; +use dom::element::{AttributeMutation, ElementTypeId, RawLayoutElementHelpers}; use dom::eventtarget::{EventTarget, EventTargetTypeId}; use dom::htmlelement::{HTMLElement, HTMLElementTypeId}; use dom::node::{Node, NodeTypeId}; use dom::virtualmethods::VirtualMethods; use std::cell::Cell; use string_cache::Atom; -use util::str::{self, DOMString}; +use style::values::specified; +use util::str::{self, DOMString, parse_legacy_font_size}; #[dom_struct] pub struct HTMLFontElement { @@ -64,6 +65,12 @@ impl HTMLFontElementMethods for HTMLFontElement { // https://html.spec.whatwg.org/multipage/#dom-font-face make_atomic_setter!(SetFace, "face"); + + // https://html.spec.whatwg.org/multipage/#dom-font-size + make_getter!(Size); + + // https://html.spec.whatwg.org/multipage/#dom-font-size + make_setter!(SetSize, "size"); } impl VirtualMethods for HTMLFontElement { @@ -92,6 +99,10 @@ impl VirtualMethods for HTMLFontElement { fn parse_plain_attribute(&self, name: &Atom, value: DOMString) -> AttrValue { match name { &atom!("face") => AttrValue::from_atomic(value), + &atom!("size") => { + let length = parse_legacy_font_size(&value).and_then(|parsed| specified::Length::from_str(&parsed)); + AttrValue::Length(value, length) + }, _ => self.super_type().unwrap().parse_plain_attribute(name, value), } } @@ -111,4 +122,14 @@ impl HTMLFontElement { None => None, } } + + #[allow(unsafe_code)] + pub fn get_size(&self) -> Option<specified::Length> { + unsafe { + ElementCast::from_ref(self) + .get_attr_for_layout(&ns!(""), &atom!("size")) + .and_then(AttrValue::as_length) + .cloned() + } + } } diff --git a/components/script/dom/webidls/HTMLFontElement.webidl b/components/script/dom/webidls/HTMLFontElement.webidl index 8419fcab10f..2b718558513 100644 --- a/components/script/dom/webidls/HTMLFontElement.webidl +++ b/components/script/dom/webidls/HTMLFontElement.webidl @@ -7,5 +7,5 @@ interface HTMLFontElement : HTMLElement { [TreatNullAs=EmptyString] attribute DOMString color; attribute DOMString face; - // attribute DOMString size; + attribute DOMString size; }; diff --git a/components/style/properties.mako.rs b/components/style/properties.mako.rs index 9bf6da3b891..86d553871b8 100644 --- a/components/style/properties.mako.rs +++ b/components/style/properties.mako.rs @@ -1919,6 +1919,7 @@ pub mod longhands { use app_units::Au; use cssparser::ToCss; use std::fmt; + use values::FONT_MEDIUM_PX; use values::computed::Context; impl ToCss for SpecifiedValue { @@ -1933,9 +1934,8 @@ pub mod longhands { use app_units::Au; pub type T = Au; } - const MEDIUM_PX: i32 = 16; #[inline] pub fn get_initial_value() -> computed_value::T { - Au::from_px(MEDIUM_PX) + Au::from_px(FONT_MEDIUM_PX) } impl ToComputedValue for SpecifiedValue { @@ -1958,21 +1958,8 @@ pub mod longhands { specified::LengthOrPercentage::Calc(_) => Err(()) }) .or_else(|()| { - match_ignore_ascii_case! { try!(input.expect_ident()), - "xx-small" => Ok(specified::Length::Absolute(Au::from_px(MEDIUM_PX) * 3 / 5)), - "x-small" => Ok(specified::Length::Absolute(Au::from_px(MEDIUM_PX) * 3 / 4)), - "small" => Ok(specified::Length::Absolute(Au::from_px(MEDIUM_PX) * 8 / 9)), - "medium" => Ok(specified::Length::Absolute(Au::from_px(MEDIUM_PX))), - "large" => Ok(specified::Length::Absolute(Au::from_px(MEDIUM_PX) * 6 / 5)), - "x-large" => Ok(specified::Length::Absolute(Au::from_px(MEDIUM_PX) * 3 / 2)), - "xx-large" => Ok(specified::Length::Absolute(Au::from_px(MEDIUM_PX) * 2)), - - // https://github.com/servo/servo/issues/3423#issuecomment-56321664 - "smaller" => Ok(specified::Length::FontRelative(specified::FontRelativeLength::Em(0.85))), - "larger" => Ok(specified::Length::FontRelative(specified::FontRelativeLength::Em(1.2))) - - _ => Err(()) - } + let ident = try!(input.expect_ident()); + specified::Length::from_str(&ident as &str).ok_or(()) }) .map(SpecifiedValue) } diff --git a/components/style/values.rs b/components/style/values.rs index 47ce425232b..d8850dc31cf 100644 --- a/components/style/values.rs +++ b/components/style/values.rs @@ -66,6 +66,8 @@ macro_rules! define_numbered_css_keyword_enum { pub type CSSFloat = f32; +pub const FONT_MEDIUM_PX: i32 = 16; + pub mod specified { use app_units::Au; @@ -79,7 +81,7 @@ pub mod specified { use std::ops::Mul; use style_traits::values::specified::AllowedNumericType; use super::AuExtensionMethods; - use super::CSSFloat; + use super::{CSSFloat, FONT_MEDIUM_PX}; use url::Url; #[derive(Clone, PartialEq, Debug, HeapSizeOf)] @@ -293,6 +295,24 @@ pub mod specified { const AU_PER_PT: CSSFloat = AU_PER_IN / 72.; const AU_PER_PC: CSSFloat = AU_PER_PT * 12.; impl Length { + // https://drafts.csswg.org/css-fonts-3/#font-size-prop + pub fn from_str(s: &str) -> Option<Length> { + Some(match_ignore_ascii_case! { s, + "xx-small" => Length::Absolute(Au::from_px(FONT_MEDIUM_PX) * 3 / 5), + "x-small" => Length::Absolute(Au::from_px(FONT_MEDIUM_PX) * 3 / 4), + "small" => Length::Absolute(Au::from_px(FONT_MEDIUM_PX) * 8 / 9), + "medium" => Length::Absolute(Au::from_px(FONT_MEDIUM_PX)), + "large" => Length::Absolute(Au::from_px(FONT_MEDIUM_PX) * 6 / 5), + "x-large" => Length::Absolute(Au::from_px(FONT_MEDIUM_PX) * 3 / 2), + "xx-large" => Length::Absolute(Au::from_px(FONT_MEDIUM_PX) * 2), + + // https://github.com/servo/servo/issues/3423#issuecomment-56321664 + "smaller" => Length::FontRelative(FontRelativeLength::Em(0.85)), + "larger" => Length::FontRelative(FontRelativeLength::Em(1.2)) + _ => return None + }) + } + #[inline] fn parse_internal(input: &mut Parser, context: &AllowedNumericType) -> Result<Length, ()> { match try!(input.next()) { diff --git a/components/util/str.rs b/components/util/str.rs index e2759e936d6..ef84853493c 100644 --- a/components/util/str.rs +++ b/components/util/str.rs @@ -9,7 +9,7 @@ use num_lib::ToPrimitive; use std::ascii::AsciiExt; use std::borrow::ToOwned; use std::ffi::CStr; -use std::iter::Filter; +use std::iter::{Filter, Peekable}; use std::ops::Deref; use std::str::{FromStr, Split, from_utf8}; @@ -47,17 +47,37 @@ pub fn split_html_space_chars<'a>(s: &'a str) -> s.split(HTML_SPACE_CHARACTERS).filter(not_empty as fn(&&str) -> bool) } + +fn is_ascii_digit(c: &char) -> bool { + match *c { + '0'...'9' => true, + _ => false, + } +} + + +fn read_numbers<I: Iterator<Item=char>>(mut iter: Peekable<I>) -> Option<i64> { + match iter.peek() { + Some(c) if is_ascii_digit(c) => (), + _ => return None, + } + + iter.take_while(is_ascii_digit).map(|d| { + d as i64 - '0' as i64 + }).fold(Some(0i64), |accumulator, d| { + accumulator.and_then(|accumulator| { + accumulator.checked_mul(10) + }).and_then(|accumulator| { + accumulator.checked_add(d) + }) + }) +} + + /// Shared implementation to parse an integer according to /// <https://html.spec.whatwg.org/#rules-for-parsing-integers> or /// <https://html.spec.whatwg.org/#rules-for-parsing-non-negative-integers> fn do_parse_integer<T: Iterator<Item=char>>(input: T) -> Option<i64> { - fn is_ascii_digit(c: &char) -> bool { - match *c { - '0'...'9' => true, - _ => false, - } - } - let mut input = input.skip_while(|c| { HTML_SPACE_CHARACTERS.iter().any(|s| s == c) }).peekable(); @@ -75,20 +95,7 @@ fn do_parse_integer<T: Iterator<Item=char>>(input: T) -> Option<i64> { Some(_) => 1, }; - match input.peek() { - Some(c) if is_ascii_digit(c) => (), - _ => return None, - } - - let value = input.take_while(is_ascii_digit).map(|d| { - d as i64 - '0' as i64 - }).fold(Some(0i64), |accumulator, d| { - accumulator.and_then(|accumulator| { - accumulator.checked_mul(10) - }).and_then(|accumulator| { - accumulator.checked_add(d) - }) - }); + let value = read_numbers(input); return value.and_then(|value| value.checked_mul(sign)); } @@ -166,6 +173,61 @@ pub fn parse_length(mut value: &str) -> LengthOrPercentageOrAuto { } } +/// https://html.spec.whatwg.org/multipage/#rules-for-parsing-a-legacy-font-size +pub fn parse_legacy_font_size(mut input: &str) -> Option<&'static str> { + // Steps 1 & 2 are not relevant + + // Step 3 + input = input.trim_matches(WHITESPACE); + + enum ParseMode { + RelativePlus, + RelativeMinus, + Absolute, + } + let mut input_chars = input.chars().peekable(); + let parse_mode = match input_chars.peek() { + // Step 4 + None => return None, + + // Step 5 + Some(&'+') => { + let _ = input_chars.next(); // consume the '+' + ParseMode::RelativePlus + } + Some(&'-') => { + let _ = input_chars.next(); // consume the '-' + ParseMode::RelativeMinus + } + Some(_) => ParseMode::Absolute, + }; + + // Steps 6, 7, 8 + let mut value = match read_numbers(input_chars) { + Some(v) => v, + None => return None, + }; + + // Step 9 + match parse_mode { + ParseMode::RelativePlus => value = 3 + value, + ParseMode::RelativeMinus => value = 3 - value, + ParseMode::Absolute => (), + } + + // Steps 10, 11, 12 + Some(match value { + n if n >= 7 => "xxx-large", + 6 => "xx-large", + 5 => "x-large", + 4 => "large", + 3 => "medium", + 2 => "small", + n if n <= 1 => "x-small", + _ => unreachable!(), + }) +} + /// Parses a legacy color per HTML5 § 2.4.6. If unparseable, `Err` is returned. pub fn parse_legacy_color(mut input: &str) -> Result<RGBA, ()> { // Steps 1 and 2. diff --git a/tests/wpt/metadata/html/dom/interfaces.html.ini b/tests/wpt/metadata/html/dom/interfaces.html.ini index fdc456b024d..066606a1e71 100644 --- a/tests/wpt/metadata/html/dom/interfaces.html.ini +++ b/tests/wpt/metadata/html/dom/interfaces.html.ini @@ -8640,12 +8640,6 @@ [HTMLFontElement interface: existence and properties of interface object] expected: FAIL - [HTMLFontElement interface: attribute size] - expected: FAIL - - [HTMLFontElement interface: document.createElement("font") must inherit property "size" with the proper type (2)] - expected: FAIL - [Window interface: window must inherit property "close" with the proper type (13)] expected: FAIL diff --git a/tests/wpt/metadata/html/dom/reflection-obsolete.html.ini b/tests/wpt/metadata/html/dom/reflection-obsolete.html.ini index b971c6c097d..f81b0e937f2 100644 --- a/tests/wpt/metadata/html/dom/reflection-obsolete.html.ini +++ b/tests/wpt/metadata/html/dom/reflection-obsolete.html.ini @@ -8523,135 +8523,6 @@ [font.tabIndex: IDL set to -2147483648 followed by getAttribute()] expected: FAIL - [font.size: typeof IDL attribute] - expected: FAIL - - [font.size: IDL get with DOM attribute unset] - expected: FAIL - - [font.size: setAttribute() to "" followed by IDL get] - expected: FAIL - - [font.size: setAttribute() to " \\0\\x01\\x02\\x03\\x04\\x05\\x06\\x07 \\b\\t\\n\\v\\f\\r\\x0e\\x0f \\x10\\x11\\x12\\x13\\x14\\x15\\x16\\x17 \\x18\\x19\\x1a\\x1b\\x1c\\x1d\\x1e\\x1f foo " followed by IDL get] - expected: FAIL - - [font.size: setAttribute() to undefined followed by IDL get] - expected: FAIL - - [font.size: setAttribute() to 7 followed by IDL get] - expected: FAIL - - [font.size: setAttribute() to 1.5 followed by IDL get] - expected: FAIL - - [font.size: setAttribute() to true followed by IDL get] - expected: FAIL - - [font.size: setAttribute() to false followed by IDL get] - expected: FAIL - - [font.size: setAttribute() to object "[object Object\]" followed by IDL get] - expected: FAIL - - [font.size: setAttribute() to NaN followed by IDL get] - expected: FAIL - - [font.size: setAttribute() to Infinity followed by IDL get] - expected: FAIL - - [font.size: setAttribute() to -Infinity followed by IDL get] - expected: FAIL - - [font.size: setAttribute() to "\\0" followed by IDL get] - expected: FAIL - - [font.size: setAttribute() to null followed by IDL get] - expected: FAIL - - [font.size: setAttribute() to object "test-toString" followed by IDL get] - expected: FAIL - - [font.size: setAttribute() to object "test-valueOf" followed by IDL get] - expected: FAIL - - [font.size: IDL set to "" followed by getAttribute()] - expected: FAIL - - [font.size: IDL set to " \\0\\x01\\x02\\x03\\x04\\x05\\x06\\x07 \\b\\t\\n\\v\\f\\r\\x0e\\x0f \\x10\\x11\\x12\\x13\\x14\\x15\\x16\\x17 \\x18\\x19\\x1a\\x1b\\x1c\\x1d\\x1e\\x1f foo " followed by getAttribute()] - expected: FAIL - - [font.size: IDL set to undefined followed by getAttribute()] - expected: FAIL - - [font.size: IDL set to undefined followed by IDL get] - expected: FAIL - - [font.size: IDL set to 7 followed by getAttribute()] - expected: FAIL - - [font.size: IDL set to 7 followed by IDL get] - expected: FAIL - - [font.size: IDL set to 1.5 followed by getAttribute()] - expected: FAIL - - [font.size: IDL set to 1.5 followed by IDL get] - expected: FAIL - - [font.size: IDL set to true followed by getAttribute()] - expected: FAIL - - [font.size: IDL set to true followed by IDL get] - expected: FAIL - - [font.size: IDL set to false followed by getAttribute()] - expected: FAIL - - [font.size: IDL set to false followed by IDL get] - expected: FAIL - - [font.size: IDL set to object "[object Object\]" followed by getAttribute()] - expected: FAIL - - [font.size: IDL set to object "[object Object\]" followed by IDL get] - expected: FAIL - - [font.size: IDL set to NaN followed by getAttribute()] - expected: FAIL - - [font.size: IDL set to NaN followed by IDL get] - expected: FAIL - - [font.size: IDL set to Infinity followed by getAttribute()] - expected: FAIL - - [font.size: IDL set to Infinity followed by IDL get] - expected: FAIL - - [font.size: IDL set to -Infinity followed by getAttribute()] - expected: FAIL - - [font.size: IDL set to -Infinity followed by IDL get] - expected: FAIL - - [font.size: IDL set to "\\0" followed by getAttribute()] - expected: FAIL - - [font.size: IDL set to null followed by getAttribute()] - expected: FAIL - - [font.size: IDL set to null followed by IDL get] - expected: FAIL - - [font.size: IDL set to object "test-toString" followed by getAttribute()] - expected: FAIL - - [font.size: IDL set to object "test-toString" followed by IDL get] - expected: FAIL - - [font.size: IDL set to object "test-valueOf" followed by IDL get] - expected: FAIL - [font.itemScope: typeof IDL attribute] expected: FAIL diff --git a/tests/wpt/mozilla/meta/MANIFEST.json b/tests/wpt/mozilla/meta/MANIFEST.json index 5f0453b175e..29a8993bc7a 100644 --- a/tests/wpt/mozilla/meta/MANIFEST.json +++ b/tests/wpt/mozilla/meta/MANIFEST.json @@ -647,6 +647,12 @@ "url": "/_mozilla/mozilla/htmlfieldsetelement_elements.html" } ], + "mozilla/htmlfontelement_size_attribute.html": [ + { + "path": "mozilla/htmlfontelement_size_attribute.html", + "url": "/_mozilla/mozilla/htmlfontelement_size_attribute.html" + } + ], "mozilla/htmlspacechars.html": [ { "path": "mozilla/htmlspacechars.html", diff --git a/tests/wpt/mozilla/tests/mozilla/htmlfontelement_size_attribute.html b/tests/wpt/mozilla/tests/mozilla/htmlfontelement_size_attribute.html new file mode 100644 index 00000000000..6a17e70e24b --- /dev/null +++ b/tests/wpt/mozilla/tests/mozilla/htmlfontelement_size_attribute.html @@ -0,0 +1,34 @@ +<!DOCTYPE html> +<meta charset=utf-8> +<title>'font' element, 'size' attribute</title> +<link rel=help href="https://html.spec.whatwg.org/multipage/#dom-font-size"> +<link rel=help href="https://html.spec.whatwg.org/multipage/#rules-for-parsing-a-legacy-font-size"> +<link rel=help href="https://drafts.csswg.org/css-fonts-3/#font-size-prop"> +<link rel=author title="Corey Farwell" href="mailto:coreyf@rwell.org"> +<script src="/resources/testharness.js"></script> +<script src="/resources/testharnessreport.js"></script> +<div id="log"></div> + +<font>Hello</font> + +<script> +var elem = document.getElementsByTagName('font')[0]; + +var modes = ["", "+", "-"]; +var sizes = ["0", "1", "2", "3", "4", "5", "6", "7", "8"]; + +var testSize = function (attrValue) { + elem.setAttribute("size", attrValue); + assert_equals(elem.getAttribute("size"), attrValue); +} + +var args = []; +for (var i = 0; i < modes.length; i++) { + for (var j = 0; j < sizes.length; j++) { + var sizeAttrValue = modes[i] + sizes[j]; + args.push(["size=" + sizeAttrValue, sizeAttrValue]); + } +} + +generate_tests(testSize, args); +</script> |