Chromium Code Reviews Chromium Code Reviews
Issue 1439333002:
Service Worker: Add Clients.get(id) (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: third_party/WebKit/LayoutTests/http/tests/serviceworker/clients-get.html |
| diff --git a/third_party/WebKit/LayoutTests/http/tests/serviceworker/clients-get.html b/third_party/WebKit/LayoutTests/http/tests/serviceworker/clients-get.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..99fefbf468b0c78627a8d609bbce62409e66903b |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/http/tests/serviceworker/clients-get.html |
| @@ -0,0 +1,67 @@ |
| +<!DOCTYPE html> |
| +<title>Service Worker: Clients.get</title> |
| +<script src="../resources/testharness.js"></script> |
| +<script src="../resources/testharnessreport.js"></script> |
| +<script src="../resources/get-host-info.js"></script> |
| +<script src="resources/test-helpers.js"></script> |
| +<script> |
| +var scope = 'resources/clients-get-frame.html'; |
| +var client_ids = []; |
| +var frame; |
| +promise_test(function(t) { |
| + return service_worker_unregister_and_register( |
| + t, 'resources/clients-get-worker.js', scope) |
| + .then(function(registration) { |
| + add_completion_callback(function() { registration.unregister(); }); |
| + return wait_for_state(t, registration.installing, 'activated'); |
| + }) |
| + .then(function() { |
| + return with_iframe(scope + '#1'); |
| + }) |
| + .then(function(frame1) { |
| + add_completion_callback(function() { frame1.remove(); }); |
| + frame1.focus(); |
| + return wait_for_clientId(); |
| + }) |
| + .then(function(client_id) { |
| + client_ids.push(client_id); |
| + return with_iframe(scope + '#2'); |
| + }) |
| + .then(function(frame2) { |
| + frame = frame2; |
| + add_completion_callback(function() { frame2.remove(); }); |
| + return wait_for_clientId(); |
| + }) |
| + .then(function(client_id) { |
| + client_ids.push(client_id, 'invalid-id'); |
| + var channel = new MessageChannel(); |
| + channel.port1.onmessage = t.step_func(on_message); |
|
nhiroki
2016/02/19 05:18:31
This test may finish before |on_message| is fired.
jungkees
2016/02/19 16:31:43
That's right. I'm working on this. Actually having
jungkees
2016/02/22 05:23:56
It was not an error in this test case. GetCallback
jungkees
2016/02/22 14:05:09
Promisified the block and fixed the renderer crash
|
| + frame.contentWindow.navigator.serviceWorker.controller.postMessage( |
| + {port: channel.port2, clientIds: client_ids}, [channel.port2]); |
| + }); |
| + }, 'Test Clients.get()'); |
| + |
| +function wait_for_clientId() { |
| + return new Promise(function(resolve, reject) { |
| + function get_client_id(e) { |
| + window.removeEventListener('message', get_client_id); |
| + resolve(e.data.clientId); |
| + } |
| + window.addEventListener('message', get_client_id, false); |
| + }); |
| +} |
| + |
| +var expected = [ |
| + /* visibilityState, focused, url, frameType */ |
| + ['visible', true, normalizeURL(scope) + '#1', 'nested'], |
| + ['visible', false, normalizeURL(scope) + '#2', 'nested'], |
| + undefined |
| +]; |
| + |
| +function on_message(e) { |
| + assert_equals(e.data.length, 3); |
| + assert_array_equals(e.data[0], expected[0]); |
| + assert_array_equals(e.data[1], expected[1]); |
| + assert_equals(e.data[2], expected[2]); |
| +} |
| +</script> |
