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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium/resources/clients-openwindow.js

Issue 1505023004: ServiceWorker: Early reject error if url is something wrong. Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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
Index: third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium/resources/clients-openwindow.js
diff --git a/third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium/resources/clients-openwindow.js b/third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium/resources/clients-openwindow.js
index 0db803c7c4e86fe8db7713046536e7ca174e24c7..68e7e148a194f9ca580077a3cfcd7f082b4b6c93 100644
--- a/third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium/resources/clients-openwindow.js
+++ b/third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium/resources/clients-openwindow.js
@@ -49,18 +49,18 @@ var TESTS = [
function testOpenAboutBlank() {
synthesizeNotificationClick().then(function(e) {
- clients.openWindow('about:blank').then(function(c) {
+ clients.openWindow('about:blank').catch(function(error) {
self.postMessage('openWindow() can open about:blank');
- self.postMessage('openWindow() result: ' + c);
+ self.postMessage('openWindow() error is: ' + error.name);
}).then(runNextTestOrQuit);
});
},
function testOpenAboutCrash() {
synthesizeNotificationClick().then(function(e) {
- clients.openWindow('about:crash').then(function(c) {
+ clients.openWindow('about:crash').catch(function(error) {
self.postMessage('openWindow() can open about:crash');
- self.postMessage('openWindow() result: ' + c);
+ self.postMessage('openWindow() error is: ' + error.name);
}).then(runNextTestOrQuit);
});
},

Powered by Google App Engine
This is Rietveld 408576698