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

Unified Diff: LayoutTests/storage/indexeddb/resources/index-multientry.js

Issue 307653004: IndexedDB: Result of failed request should be |undefined| (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 7 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/storage/indexeddb/resources/index-multientry.js
diff --git a/LayoutTests/storage/indexeddb/resources/index-multientry.js b/LayoutTests/storage/indexeddb/resources/index-multientry.js
index 71b96a8e68c61192ad73d6b971f31dbfabb64d13..72c0cce13d43cc885c1bc8aa85571b0eb17e7f20 100644
--- a/LayoutTests/storage/indexeddb/resources/index-multientry.js
+++ b/LayoutTests/storage/indexeddb/resources/index-multientry.js
@@ -99,7 +99,11 @@ function verifyUniqueConstraint()
debug("This should fail the uniqueness constraint on the index, and fail:");
request = evalAndLog("transaction.objectStore('store-unique').put({x: [5, 2], y: 'c'}, 'should fail')");
request.onsuccess = unexpectedSuccessCallback;
- request.onerror = function() { debug("Request failed, as expected (" + request.error.name + ")"); };
+ request.onerror = function() {
+ shouldBeUndefined("request.result");
+ shouldBeNonNull("request.error");
+ debug("Request failed, as expected (" + request.error.name + ")");
+ };
};
};
}

Powered by Google App Engine
This is Rietveld 408576698