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

Unified Diff: LayoutTests/fast/xsl/xslt-processor.html

Issue 795053002: [binding] Migrate XSLTProcessor.idl away from using custom V8 binding. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Review comments Created 6 years 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
« no previous file with comments | « no previous file | LayoutTests/fast/xsl/xslt-processor-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/fast/xsl/xslt-processor.html
diff --git a/LayoutTests/fast/xsl/xslt-processor.html b/LayoutTests/fast/xsl/xslt-processor.html
index 2bd7a4efdc7569012e21e0bb3a65639fbbabcd91..aae8fc1213837bdd26a33563149a13dafb689235 100644
--- a/LayoutTests/fast/xsl/xslt-processor.html
+++ b/LayoutTests/fast/xsl/xslt-processor.html
@@ -219,12 +219,14 @@ addResultExpectValue("pass undefined namespace", processor.getParameter(undefine
// 4.3 pass setParameter an undefined name
var testname = "pass setParameter an undefined name";
-addResultExpectException(testname, function () { return processor.setParameter(undefined, undefined , "Success"); } );
+processor.setParameter(undefined, undefined , "Success");
+addResultExpectValue(testname, processor.getParameter(undefined, undefined), "Success");
// 4.4 pass undefined value
processor.setParameter(undefined, "foo", undefined);
-var testname = "pass undefined value";
+var testname = "pass undefined value to setParameter";
+addResultExpectValue(testname, processor.getParameter(undefined, "foo"), "undefined");
// 4.5 pass unsupported value to setParameter (object, for instance?)
@@ -239,7 +241,7 @@ addSectionHeader("Value getParameter(in DOMString namespaceURI, in DOMString loc
// 5.1 pass undefined name
var testname = "pass getParameter an undefined name";
-addResultExpectException(testname, function () { return processor.getParameter(undefined, undefined); } );
+addResultExpectValue(testname, processor.getParameter(undefined, undefined), "Success");
// 5.2 pass getParameter a name which has not been set
@@ -274,7 +276,7 @@ addResultExpectValue(testname, processor.getParameter(null, "test"), undefined);
processor.setParameter(undefined, "undefined", "Success");
processor.removeParameter(undefined, undefined);
var testname = "pass removeParameter undefined name";
-addResultExpectValue(testname, processor.getParameter(undefined, "undefined"), "Success");
+addResultExpectValue(testname, processor.getParameter(undefined, "undefined"), undefined);
// 6.4 pass removeParameter a name which has not been set
« no previous file with comments | « no previous file | LayoutTests/fast/xsl/xslt-processor-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698