Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(31)

Unified Diff: LayoutTests/fast/forms/fieldset/fieldset-name.html

Issue 70103011: Remove TreatNullAs=NullString for HTMLFieldSetElement (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: update tests Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/fast/forms/fieldset/fieldset-name.html
diff --git a/LayoutTests/fast/forms/fieldset/fieldset-name.html b/LayoutTests/fast/forms/fieldset/fieldset-name.html
index 63f828dac24e18d6bd30f836c1bdc66651e4dba9..d7b30bb88625dc9f309af088de361625810051ec 100644
--- a/LayoutTests/fast/forms/fieldset/fieldset-name.html
+++ b/LayoutTests/fast/forms/fieldset/fieldset-name.html
@@ -16,13 +16,11 @@ 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")');
+shouldBe('fs1.name', '"null"');
+shouldBe('fs1.getAttribute("name")', '"null"');
+shouldBeTrue('fs1.hasAttribute("name")');
debug('fs1.name=undefined');
fs1.name = undefined;

Powered by Google App Engine
This is Rietveld 408576698