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

Unified Diff: third_party/WebKit/LayoutTests/storage/indexeddb/resources/intversion-bad-parameters.js

Issue 1476863003: bindings: Ignores the last undefined arguments when counting the args. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed a review comment. Created 5 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: third_party/WebKit/LayoutTests/storage/indexeddb/resources/intversion-bad-parameters.js
diff --git a/third_party/WebKit/LayoutTests/storage/indexeddb/resources/intversion-bad-parameters.js b/third_party/WebKit/LayoutTests/storage/indexeddb/resources/intversion-bad-parameters.js
index 3853fc3cabdc7017c1e40957e2725addf26903e3..062a7f02356d80c942b0c4fca048153b6355bfd8 100644
--- a/third_party/WebKit/LayoutTests/storage/indexeddb/resources/intversion-bad-parameters.js
+++ b/third_party/WebKit/LayoutTests/storage/indexeddb/resources/intversion-bad-parameters.js
@@ -25,7 +25,9 @@ function deleteSuccess(evt) {
evalAndExpectExceptionClass("indexedDB.open(dbname, -1)", "TypeError");
evalAndExpectExceptionClass("indexedDB.open(dbname, 0x20000000000000)", "TypeError");
evalAndExpectExceptionClass("indexedDB.open(dbname, null)", "TypeError");
- evalAndExpectExceptionClass("indexedDB.open(dbname, undefined)", "TypeError");
+ // Unlike open(dbname, null), passing |undefined| to the second parameter
+ // (version) is okay because the version is an optional parameter.
+ shouldNotThrow("indexedDB.open(dbname, undefined)");
finishJSTest();
}

Powered by Google App Engine
This is Rietveld 408576698