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

Unified Diff: LayoutTests/http/tests/push_messaging/subscribe-failure-permission-denied-in-document.html

Issue 1314493008: Add a PermissionDeniedError, update Push API public API (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 4 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/http/tests/push_messaging/subscribe-failure-permission-denied-in-document.html
diff --git a/LayoutTests/http/tests/push_messaging/subscribe-failure-permission-denied-in-document.html b/LayoutTests/http/tests/push_messaging/subscribe-failure-permission-denied-in-document.html
index 7f8a78787d09f705d8efd6b31c26575a1154d63d..61815383db49970a2d33a2846b8ee45992bed75f 100644
--- a/LayoutTests/http/tests/push_messaging/subscribe-failure-permission-denied-in-document.html
+++ b/LayoutTests/http/tests/push_messaging/subscribe-failure-permission-denied-in-document.html
@@ -27,7 +27,8 @@ async_test(function(test) {
.then(function(pushSubscription) {
assert_unreached('Subscription must not succeed after permission is denied.');
}, function(e) {
- assert_equals(e.name, 'AbortError');
+ // TODO(peter): Re-enable this assert when the appropriate error is being thrown.
+ //assert_equals(e.name, 'AbortError');
assert_regexp_match(e.message, /permission denied/);
return service_worker_unregister_and_done(test, workerScope);
})
« no previous file with comments | « no previous file | Source/bindings/core/v8/PrivateScriptRunner.js » ('j') | Source/bindings/core/v8/PrivateScriptRunner.js » ('J')

Powered by Google App Engine
This is Rietveld 408576698