ServiceWorker: Rewrite windowclient-navigate.html test.

third_party/WebKit/LayoutTests/http/tests/serviceworker/resources/navigate-on-install-worker.js

Index: third_party/WebKit/LayoutTests/http/tests/serviceworker/resources/navigate-on-install-worker.js
diff --git a/third_party/WebKit/LayoutTests/http/tests/serviceworker/resources/navigate-on-install-worker.js b/third_party/WebKit/LayoutTests/http/tests/serviceworker/resources/navigate-on-install-worker.js
new file mode 100644
index 0000000000000000000000000000000000000000..d64c6766eaa99134e69bbec6c129f82696bc403a
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/http/tests/serviceworker/resources/navigate-on-install-worker.js
@@ -0,0 +1,4 @@
+self.addEventListener('install', function(e) {
+  importScripts('windowclient-navigate-worker.js');

[nhiroki, 2016/01/25 09:50:31]

I think importing non-library file (other worker file) would not be really
maintainable and side-effect is likely to slip in. For example,
get_message/client_navigate might be called twice at line 3 in this file and at
line 1 in windowclient-navigate-worker.js. I guess this is not expected behavior
(right?).

Can we merge this event handler into "windowclient-navigate-worker.js" and
execute it when an optional query parameter is specified? For example, if script
url "windowclient-navigate-worker.js" has a query parameter
"navigate-on-install", run client_navigate in the install event.

[zino, 2016/01/29 05:36:38]

Done.

+  e.waitUntil(get_message().then(client_navigate));

[nhiroki, 2016/01/25 09:50:31]

Let me confirm... you mean to test the case where a non-active worker attempts
to navigate an in-scope client. Is this right? Can we also test the case where
an active worker attempts to navigate an in-scope-but-not-controlled-yet client?

[zino, 2016/01/29 05:36:38]

I tried it but I couldn't yet.
I'm not sure but there is some bug when using e.waitUntil in activate event.
I'll upload a minified test script to another CL soon. I need your help.

[nhiroki, 2016/02/01 07:13:55]

Commented in the patchset 5.

+});