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

Unified Diff: LayoutTests/http/tests/serviceworker/resources/fetch-mixed-content-iframe.html

Issue 466723002: ServiceWorker: Enable ServiceWorkerRegistration and update layout tests (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@updatefound
Patch Set: apply horo@'s patch (PatchSet 28 on https://codereview.chromium.org/468753003/) Created 6 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/serviceworker/resources/fetch-mixed-content-iframe.html
diff --git a/LayoutTests/http/tests/serviceworker/resources/fetch-mixed-content-iframe.html b/LayoutTests/http/tests/serviceworker/resources/fetch-mixed-content-iframe.html
index 244972ce67513da71f834e3ff1e7725ddcf50070..ec9612df3d683d920717232cc58f5753316e973c 100644
--- a/LayoutTests/http/tests/serviceworker/resources/fetch-mixed-content-iframe.html
+++ b/LayoutTests/http/tests/serviceworker/resources/fetch-mixed-content-iframe.html
@@ -11,6 +11,13 @@ navigator.serviceWorker.unregister(SCOPE)
.then(function() {
return navigator.serviceWorker.register(URL, {scope: SCOPE});
})
+ .then(function(registration) {
+ return new Promise(function(resolve, reject) {
+ registration.addEventListener('updatefound', function() {
+ resolve(registration.installing);
falken 2014/08/14 10:16:36 this promise is just wait_for_update()?
nhiroki 2014/08/14 13:22:07 I think we cannot call wait_for_update() here sinc
+ });
+ });
+ })
.then(function(worker) {
worker.addEventListener('statechange', on_state_change);
});

Powered by Google App Engine
This is Rietveld 408576698