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

Unified Diff: LayoutTests/http/tests/serviceworker/install-event-type.html

Issue 1067513002: Removing ServiceWorker's InstallEvent Interface. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 years, 8 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/serviceworker/install-event-type.html
diff --git a/LayoutTests/http/tests/serviceworker/install-event-type.html b/LayoutTests/http/tests/serviceworker/install-event-type.html
new file mode 100644
index 0000000000000000000000000000000000000000..dabe49a2b4f360bb7bf906ab7ff140598be3a12b
--- /dev/null
+++ b/LayoutTests/http/tests/serviceworker/install-event-type.html
@@ -0,0 +1,29 @@
+<!DOCTYPE html>
+<script src="../resources/testharness.js"></script>
+<script src="../resources/testharness-helpers.js"></script>
+<script src="../resources/testharnessreport.js"></script>
+<script src="resources/test-helpers.js"></script>
+<script>
+function wait_for_install_event(worker) {
+ return new Promise(function(resolve) {
+ worker.addEventListener('statechange', function(event) {
+ if (worker.state == 'installed')
+ resolve(true);
+ else if (worker.state == 'redundant')
+ resolve(false);
+ });
+ });
+}
+
+promise_test(function(t) {
+ return service_worker_unregister_and_register(t, 'resources/oninstall-event-type.js', 'resources/oninstall-event-type.js')
falken 2015/04/08 05:44:46 By convention, service worker script filenames end
Paritosh Kumar 2015/04/08 06:44:06 Thanks. Done. OO, I was using 'resources/oninstal
+ .then(function(registration) {
+ return wait_for_install_event(registration.installing);
+ })
+ .then(function(did_install) {
+ assert_equals(did_install, true,'The worker was installed');
falken 2015/04/08 05:44:46 You can use assert_true rather than assert_equals
Paritosh Kumar 2015/04/08 06:44:06 Yeah.. Thanks.
+ })
+ }, 'install event type');
+
+</script>
+
falken 2015/04/08 05:44:46 nit: extra empty line
Paritosh Kumar 2015/04/08 06:44:06 Done.

Powered by Google App Engine
This is Rietveld 408576698