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

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

Issue 64173002: Align HTMLAppletElement IDL with spec (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: 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 a1bb97b9198d41b9800c157eec82c08147c82556..092082a1fc4007b48fccb87b53113143ff9be842 100644
--- a/LayoutTests/fast/dom/element-attribute-js-null.html
+++ b/LayoutTests/fast/dom/element-attribute-js-null.html
@@ -50,7 +50,7 @@
result = "<span class='fail'>TEST FAILED:</span> An exception should have been thrown.";
else if (isUrl && element[attr] === resolve(expected))
result = "<span class='pass'>TEST SUCCEEDED:</span> The value was " + resultStringifier(expected) + " resolved as a URL.";
- else if (element[attr] === expected)
+ else if (!isUrl && element[attr] === expected)
result = "<span class='pass'>TEST SUCCEEDED:</span> The value was " + resultStringifier(expected) + ".";
else
result = "<span class='fail'>TEST FAILED:</span> The value should have been " + resultStringifier(expected) + " but was " + resultStringifier(element[attr]) + ".";
@@ -124,17 +124,15 @@
type: 'HTMLAppletElement',
elementToUse: document.createElement('applet'),
attributes: [
- {name: 'align', expectedNull: ''},
- {name: 'alt', expectedNull: ''},
- {name: 'archive', expectedNull: ''},
- {name: 'code', expectedNull: ''},
- {name: 'codeBase', expectedNull: ''},
- {name: 'height', expectedNull: ''},
- {name: 'hspace', expectedNull: ''},
- {name: 'name', expectedNull: ''},
- {name: 'object', expectedNull: ''},
- {name: 'vspace', expectedNull: ''},
- {name: 'width', expectedNull: ''}
+ {name: 'align', expectedNull: 'null'},
+ {name: 'alt', expectedNull: 'null'},
+ {name: 'archive', expectedNull: 'null'},
+ {name: 'code', expectedNull: 'null'},
+ {name: 'codeBase', expectedNull: 'null', isUrl:true},
+ {name: 'height', expectedNull: 'null'},
+ {name: 'name', expectedNull: 'null'},
+ {name: 'object', expectedNull: 'null', isUrl:true},
+ {name: 'width', expectedNull: 'null'}
]
},
{

Powered by Google App Engine
This is Rietveld 408576698