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

Unified Diff: LayoutTests/fast/dom/element-attribute-js-null.html

Issue 60153008: Remove TreatNullAs=NullString for <base>, <body> and <br> (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: fix test 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/dom/element-attribute-js-null.html
diff --git a/LayoutTests/fast/dom/element-attribute-js-null.html b/LayoutTests/fast/dom/element-attribute-js-null.html
index d6d47dad6a9c11e37bafebc1e36758b4052bf84d..03ef1dff07975c95bd3be7e654659a3f1f5b65a7 100644
--- a/LayoutTests/fast/dom/element-attribute-js-null.html
+++ b/LayoutTests/fast/dom/element-attribute-js-null.html
@@ -163,7 +163,8 @@
type: 'HTMLBaseElement',
elementToUse: document.createElement('base'),
attributes: [
- {name: 'target', expectedNull: ''}
+ {name: 'href', expectedNull: 'null', isUrl: true},
+ {name: 'target', expectedNull: 'null'}
]
},
{
@@ -178,7 +179,7 @@
elementToUse: document.createElement('body'),
attributes: [
{name: 'aLink', expectedNull: ''},
- {name: 'background', expectedNull: ''},
+ {name: 'background', expectedNull: 'null'},
{name: 'bgColor', expectedNull: ''},
{name: 'link', expectedNull: ''},
{name: 'text', expectedNull: ''},
@@ -189,7 +190,7 @@
type: 'HTMLBRElement',
elementToUse: document.createElement('br'),
attributes: [
- {name: 'clear', expectedNull: ''}
+ {name: 'clear', expectedNull: 'null'}
]
},
{

Powered by Google App Engine
This is Rietveld 408576698