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

Unified Diff: LayoutTests/http/tests/background_sync/interfaces.html

Issue 963683002: Add IDL and initial Blink API for Background Sync (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Add missing serviceWorkerRegistration.syncManager to interface tests Created 5 years, 9 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
« no previous file with comments | « no previous file | LayoutTests/http/tests/background_sync/resources/empty_worker.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/http/tests/background_sync/interfaces.html
diff --git a/LayoutTests/http/tests/push_messaging/interfaces.html b/LayoutTests/http/tests/background_sync/interfaces.html
similarity index 44%
copy from LayoutTests/http/tests/push_messaging/interfaces.html
copy to LayoutTests/http/tests/background_sync/interfaces.html
index b47b3ee2dc8f49a00819157e6d8573680d6e0407..eed52248d6c4e37085ea0df6e121af2f9c20f86f 100644
--- a/LayoutTests/http/tests/push_messaging/interfaces.html
+++ b/LayoutTests/http/tests/background_sync/interfaces.html
@@ -1,37 +1,39 @@
<!doctype html>
<html>
<head>
- <title>Push API: Verifies that the right Push API interfaces get exposed.</title>
+ <title>Background Sync API: Verifies that the right Background Sync API interfaces get exposed.</title>
<script src="../resources/testharness.js"></script>
<script src="../resources/testharnessreport.js"></script>
<script src="../serviceworker/resources/test-helpers.js"></script>
</head>
<body>
<script>
- // Tests that the expected Push API interfaces are being exposed on both
+ // Tests that the expected Background Sync API interfaces are being exposed on both
// the Service Worker and Document global scopes.
service_worker_test(
'resources/interfaces-worker.js',
'Exposure of interfaces in a Service Worker.');
test(function() {
- assert_own_property(self, 'PushManager', 'PushManager needs to be exposed as a global.');
+ assert_own_property(self, 'SyncManager', 'SyncManager needs to be exposed as a global.');
- assert_own_property(PushManager.prototype, 'subscribe');
- assert_own_property(PushManager.prototype, 'getSubscription');
- assert_own_property(PushManager.prototype, 'hasPermission');
+ assert_own_property(SyncManager.prototype, 'register');
+ assert_own_property(SyncManager.prototype, 'getRegistration');
+ assert_own_property(SyncManager.prototype, 'getRegistrations');
+ // FIXME: Re-enable this once permissions are wired up
+ //assert_own_property(SyncManager.prototype, 'hasPermission');
- }, 'PushManager should be exposed and have the expected interface in a Document.');
+ }, 'SyncManager should be exposed and have the expected interface in a Document.');
test(function() {
- assert_own_property(self, 'PushSubscription', 'PushSubscription needs to be exposed as a global.');
+ assert_own_property(self, 'SyncRegistration', 'SyncRegistration needs to be exposed as a global.');
// FIXME: Assert existence of the attributes when they are properly
// exposed in the prototype chain. https://crbug.com/43394
- assert_own_property(PushSubscription.prototype, 'unsubscribe');
+ assert_own_property(SyncRegistration.prototype, 'unregister');
- }, 'PushSubscription should be exposed and have the expected interface in a Document.');
+ }, 'SyncRegistration should be exposed and have the expected interface in a Document.');
</script>
</body>
-</html>
+</html>
« no previous file with comments | « no previous file | LayoutTests/http/tests/background_sync/resources/empty_worker.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698