Implement ServiceWorkerClient attributes [3/3]

LayoutTests/http/tests/serviceworker/clients-getall.html

Index: LayoutTests/http/tests/serviceworker/clients-getall.html
diff --git a/LayoutTests/http/tests/serviceworker/clients-getall.html b/LayoutTests/http/tests/serviceworker/clients-getall.html
new file mode 100644
index 0000000000000000000000000000000000000000..52b2d59a85f76f13329967d34687e1718f3abe36
--- /dev/null
+++ b/LayoutTests/http/tests/serviceworker/clients-getall.html
@@ -0,0 +1,44 @@
+<!DOCTYPE html>
+<title>Service Worker: ServiceWorkerClient.getAll</title>
+<script src="../resources/testharness.js"></script>
+<script src="../resources/testharnessreport.js"></script>
+<script src="resources/test-helpers.js"></script>
+<script>
+var t = async_test('Test ServiceWorkerClient.getAll()');
+var scope = 'resources/blank.html';

[nhiroki, 2014/12/08 06:02:47]

"blank.html?clients-getAll" could be more stable?

[Kunihiko Sakamoto, 2014/12/08 07:40:00]

Done.

+t.step(function() {

[nhiroki, 2014/12/08 06:02:47]

Optional: You may want to use 'promise_test' and omit '.catch'.

[Kunihiko Sakamoto, 2014/12/08 07:39:59]

It didn't work as promise_test calls done() but we want to wait onMessage().

[nhiroki, 2014/12/08 08:01:08]

I think wrapping onMessage() with Promise might work (This is optional and you
don't have to change the current style)

[Kunihiko Sakamoto, 2014/12/08 08:34:10]

OK, let me keep the current style as it's easier to understand for me.

+    service_worker_unregister_and_register(
+        t, 'resources/clients-getall-worker.js', scope)
+      .then(function(registration) {
+          return wait_for_update(t, registration);

[nhiroki, 2014/12/08 06:02:47]

To get a worker object, you can just use 'registration.installing' instead of
calling 'wait_for_update()' (see https://codereview.chromium.org/776373003/ for
more details)

[Kunihiko Sakamoto, 2014/12/08 07:40:00]

Done.

+        })
+      .then(function(sw) {
+          return wait_for_state(t, sw, 'activated');
+        })
+      .then(function() { return with_iframe(scope + '#1'); })
+      .then(function(frame1) {
+          frame1.focus();
+          return with_iframe(scope + '#2');
+        })
+      .then(function(frame2) {
+          var channel = new MessageChannel();
+          channel.port1.onmessage = t.step_func(onMessage);
+          frame2.contentWindow.navigator.serviceWorker.controller.postMessage(
+              {port:channel.port2}, [channel.port2]);
+        })
+      .catch(unreached_rejection(t));
+  });
+
+var expected = [
+    /* visibilityState, focused, url, frameType */
+    ['visible', true, new URL(scope + '#1', location).toString(), 'nested'],
+    ['visible', false, new URL(scope + '#2', location).toString(), 'nested']

[nhiroki, 2014/12/08 06:02:46]

Can we have tests for other visibilityStates ('hidden' and 'preRender')?

[Kunihiko Sakamoto, 2014/12/08 07:39:59]

We could call testRunner.setPageVisibility() but that will make this test
Chromium-specific.

I've made ServiceWorkerContainerTest cover the "hidden" case.

[nhiroki, 2014/12/08 08:01:08]

Acknowledged.

+];
+
+function onMessage(e) {
+  assert_equals(e.data.length, 2);
+  assert_array_equals(e.data[0], expected[0]);
+  assert_array_equals(e.data[1], expected[1]);
+  t.done();

[nhiroki, 2014/12/08 06:02:47]

Calling service_worker_unregister_and_done() would be more polite.

[Kunihiko Sakamoto, 2014/12/08 07:40:00]

Done.

+}
+</script>