Commit ca05af39 authored by peter@chromium.org's avatar peter@chromium.org

Re-enable a push subscription-without-permission layout test.

Now that the appropriate DOMException is being thrown, we can test for that.

This CL is part of a three-sided patch:
  [1] https://codereview.chromium.org/1314493008/
  [2] https://codereview.chromium.org/1316973002/
  [3] This CL.

BUG=522206

Review URL: https://codereview.chromium.org/1313493003

git-svn-id: svn://svn.chromium.org/blink/trunk@201556 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent ee21b5ae
...@@ -27,8 +27,7 @@ async_test(function(test) { ...@@ -27,8 +27,7 @@ async_test(function(test) {
.then(function(pushSubscription) { .then(function(pushSubscription) {
assert_unreached('Subscription must not succeed after permission is denied.'); assert_unreached('Subscription must not succeed after permission is denied.');
}, function(e) { }, function(e) {
// TODO(peter): Re-enable this assert when the appropriate error is being thrown. assert_equals(e.name, 'PermissionDeniedError');
//assert_equals(e.name, 'AbortError');
assert_regexp_match(e.message, /permission denied/); assert_regexp_match(e.message, /permission denied/);
return service_worker_unregister_and_done(test, workerScope); return service_worker_unregister_and_done(test, workerScope);
}) })
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment