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

Unified Diff: LayoutTests/dom/svg/level3/xpath/XPathEvaluator_createExpression_INVALID_EXPRESSION_ERR.js

Issue 17239008: Remove XPathException (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 years, 6 months 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/dom/svg/level3/xpath/XPathEvaluator_createExpression_INVALID_EXPRESSION_ERR.js
diff --git a/LayoutTests/dom/svg/level3/xpath/XPathEvaluator_createExpression_INVALID_EXPRESSION_ERR.js b/LayoutTests/dom/svg/level3/xpath/XPathEvaluator_createExpression_INVALID_EXPRESSION_ERR.js
index 67a32362fa5ca2eb503a0b9c1817b6770770524b..3ad53b25a9f93cbaa8319b15a5448e93789fd86c 100644
--- a/LayoutTests/dom/svg/level3/xpath/XPathEvaluator_createExpression_INVALID_EXPRESSION_ERR.js
+++ b/LayoutTests/dom/svg/level3/xpath/XPathEvaluator_createExpression_INVALID_EXPRESSION_ERR.js
@@ -79,7 +79,7 @@ function loadComplete() {
'createExpression("12a", null)' method is
invoked with the document element. The method should fail to create
pre-compiled expression and throws
- XPathException.INVALID_EXPRESSION_ERR
+ SYNTAX_ERR
since "12a" is not an XPath expression.
* @author Philippe Le Hégaret
@@ -107,7 +107,7 @@ function XPathEvaluator_createExpression_INVALID_EXPRESSION_ERR() {
exp = xpEvaluator.createExpression("12a",nullNSResolver);
}
catch(ex) {
- success = (typeof(ex.code) != 'undefined' && ex.code == 51);
+ success = ex.name == 'SyntaxError';
abarth-chromium 2013/06/18 19:30:33 This looks like a W3C test. Maybe we should updat
arv (Not doing code reviews) 2013/06/18 19:34:30 I'm fine either way. Changing the expectation mean
}
assertTrue("throw_INVALID_EXPRESSION_ERR",success);
}

Powered by Google App Engine
This is Rietveld 408576698