Title: [109836] trunk
Revision
109836
Author
[email protected]
Date
2012-03-05 19:07:53 -0800 (Mon, 05 Mar 2012)

Log Message

Source/WebCore: [Forms] HTMLFieldSetForms.idl doesn't have name attribute.
https://bugs.webkit.org/show_bug.cgi?id=80108

Patch by Yoshifumi Inoue <[email protected]> on 2012-03-05
Reviewed by Hajime Morita.

Test: fast/forms/fieldset/fieldset-name.html

* html/HTMLFieldSetElement.idl: Add "name" attribute.

LayoutTests: [Forms] HTMLFieldSetForms.idl doesn't have name attribute.
https://bugs.webkit.org/show_bug.cgi?id=81008

Test read/write attribute "name" of fieldset element.

Patch by Yoshifumi Inoue <[email protected]> on 2012-03-05
Reviewed by Hajime Morita.

* fast/forms/fieldset/fieldset-name-expected.txt: Added.
* fast/forms/fieldset/fieldset-name.html: Added.

Modified Paths

Added Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (109835 => 109836)


--- trunk/LayoutTests/ChangeLog	2012-03-06 02:42:55 UTC (rev 109835)
+++ trunk/LayoutTests/ChangeLog	2012-03-06 03:07:53 UTC (rev 109836)
@@ -1,3 +1,15 @@
+2012-03-05  Yoshifumi Inoue  <[email protected]>
+
+        [Forms] HTMLFieldSetForms.idl doesn't have name attribute.
+        https://bugs.webkit.org/show_bug.cgi?id=81008
+
+        Test read/write attribute "name" of fieldset element.
+
+        Reviewed by Hajime Morita.
+
+        * fast/forms/fieldset/fieldset-name-expected.txt: Added.
+        * fast/forms/fieldset/fieldset-name.html: Added.
+
 2012-03-05  Tony Chang  <[email protected]>
 
         add tests for multiline flexbox and flex-pack

Added: trunk/LayoutTests/fast/forms/fieldset/fieldset-name-expected.txt (0 => 109836)


--- trunk/LayoutTests/fast/forms/fieldset/fieldset-name-expected.txt	                        (rev 0)
+++ trunk/LayoutTests/fast/forms/fieldset/fieldset-name-expected.txt	2012-03-06 03:07:53 UTC (rev 109836)
@@ -0,0 +1,26 @@
+This test checks fieldset.name attribute.
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+Read from HTML
+PASS fs1.name is "a"
+PASS fs1.getAttribute("name") is "a"
+fs1.name="b"
+PASS fs1.name is "b"
+PASS fs1.getAttribute("name") is "b"
+fs1.name=1234
+PASS fs1.name is "1234"
+PASS fs1.getAttribute("name") is "1234"
+fs1.name=null
+PASS fs1.name is ""
+PASS fs1.getAttribute("name") is null
+PASS fs1.hasAttribute("name") is false
+fs1.name=undefined
+PASS fs1.name is "undefined"
+PASS fs1.getAttribute("name") is "undefined"
+PASS fs1.hasAttribute("name") is true
+PASS successfullyParsed is true
+
+TEST COMPLETE
+

Added: trunk/LayoutTests/fast/forms/fieldset/fieldset-name.html (0 => 109836)


--- trunk/LayoutTests/fast/forms/fieldset/fieldset-name.html	                        (rev 0)
+++ trunk/LayoutTests/fast/forms/fieldset/fieldset-name.html	2012-03-06 03:07:53 UTC (rev 109836)
@@ -0,0 +1,34 @@
+<script src=""
+<fieldset id="fs1" name="a"></fieldset>
+<script>
+description("This test checks fieldset.name attribute.");
+debug("Read from HTML");
+shouldBe('fs1.name', '"a"');
+shouldBe('fs1.getAttribute("name")', '"a"');
+
+debug('fs1.name="b"');
+fs1.name = "b";
+shouldBe('fs1.name', '"b"');
+shouldBe('fs1.getAttribute("name")', '"b"');
+
+debug('fs1.name=1234');
+fs1.name = 1234;
+shouldBe('fs1.name', '"1234"');
+shouldBe('fs1.getAttribute("name")', '"1234"');
+
+// FIXME: The specification doesn't have clear description for setting null to name.
+// We may need to update below once the specification fixed.
+debug('fs1.name=null');
+fs1.name = null;
+shouldBe('fs1.name', '""');
+shouldBeNull('fs1.getAttribute("name")');
+shouldBeFalse('fs1.hasAttribute("name")');
+
+debug('fs1.name=undefined');
+fs1.name = undefined;
+shouldBe('fs1.name', '"undefined"');
+shouldBe('fs1.getAttribute("name")', '"undefined"');
+shouldBeTrue('fs1.hasAttribute("name")');
+</script>
+<script src=""
+

Modified: trunk/Source/WebCore/ChangeLog (109835 => 109836)


--- trunk/Source/WebCore/ChangeLog	2012-03-06 02:42:55 UTC (rev 109835)
+++ trunk/Source/WebCore/ChangeLog	2012-03-06 03:07:53 UTC (rev 109836)
@@ -1,3 +1,14 @@
+2012-03-05  Yoshifumi Inoue  <[email protected]>
+
+        [Forms] HTMLFieldSetForms.idl doesn't have name attribute.
+        https://bugs.webkit.org/show_bug.cgi?id=80108
+
+        Reviewed by Hajime Morita.
+
+        Test: fast/forms/fieldset/fieldset-name.html
+
+        * html/HTMLFieldSetElement.idl: Add "name" attribute.
+
 2012-03-05  Levi Weintraub  <[email protected]>
 
         Update usage of LayoutUnits in RenderBox

Modified: trunk/Source/WebCore/html/HTMLFieldSetElement.idl (109835 => 109836)


--- trunk/Source/WebCore/html/HTMLFieldSetElement.idl	2012-03-06 02:42:55 UTC (rev 109835)
+++ trunk/Source/WebCore/html/HTMLFieldSetElement.idl	2012-03-06 03:07:53 UTC (rev 109836)
@@ -21,6 +21,7 @@
 
     interface HTMLFieldSetElement : HTMLElement {
         readonly attribute HTMLFormElement form;
+        attribute [Reflect] DOMString name;
         readonly attribute boolean         willValidate;
         readonly attribute ValidityState   validity;
         readonly attribute DOMString       validationMessage;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to