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

Unified Diff: LayoutTests/http/tests/notifications/service-worker-show-notification-click.html

Issue 789643003: The Service Worker notificationclick event should carry a notification. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 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: LayoutTests/http/tests/notifications/service-worker-show-notification-click.html
diff --git a/LayoutTests/http/tests/notifications/service-worker-show-notification-click.html b/LayoutTests/http/tests/notifications/service-worker-show-notification-click.html
index 37ffa0c859ea42d2e5cebb40924d93b0386a61db..9154bbe5b040be277b8e30ed8a4095f68be7b178 100644
--- a/LayoutTests/http/tests/notifications/service-worker-show-notification-click.html
+++ b/LayoutTests/http/tests/notifications/service-worker-show-notification-click.html
@@ -11,6 +11,7 @@
// Tests that the showNotification() function resolves a promise, and that the
// notificationclick event gets fired on the Service Worker when we simulate a
// click on it. This test requires the test runner.
+
async_test(function (test) {
var scope = 'resources/scope/service-worker-show-notification-click',
workerUrl = 'resources/click-forward-service-worker.js';
@@ -44,8 +45,8 @@
});
}).then(function () {
messagePort.addEventListener('message', function(event) {
- assert_equals(event.data, 'The notification has been clicked on.');
- test.done();
+ assert_equals(event.data, 'Clicked on Notification: ' + scope);
+ service_worker_unregister_and_done(test, scope);
});
testRunner.simulateWebNotificationClick(scope);

Powered by Google App Engine
This is Rietveld 408576698