Chromium Code Reviews Chromium Code Reviews
Issue 1209493002:
Service Worker: Surface Client.id attribute. (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/http/tests/serviceworker/client-id.html |
| diff --git a/LayoutTests/http/tests/serviceworker/client-id.html b/LayoutTests/http/tests/serviceworker/client-id.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..91bc23266e760811e2b49e333b73d47c7e63c730 |
| --- /dev/null |
| +++ b/LayoutTests/http/tests/serviceworker/client-id.html |
| @@ -0,0 +1,63 @@ |
| +<!DOCTYPE html> |
| +<title>Service Worker: Client.id</title> |
| +<script src="../resources/testharness.js"></script> |
| +<script src="../resources/testharnessreport.js"></script> |
| +<script src="resources/test-helpers.js"></script> |
| +<script> |
| +var test; |
| +var scope = 'resources/blank.html?client-id'; |
| +var frame1, frame2; |
| + |
| +async_test(function(t) { |
| + test = t; |
| + service_worker_unregister_and_register( |
| + t, 'resources/client-id-worker.js', scope) |
| + .then(function(registration) { |
| + return wait_for_state(t, registration.installing, 'activated'); |
| + }) |
| + .then(function() { return with_iframe(scope + '#1'); }) |
| + .then(function(f) { |
| + frame1 = f; |
| + // To be sure Clients.matchAll() iterates in the same order. |
| + f.focus(); |
| + return with_iframe(scope + '#2'); |
| + }) |
| + .then(function(f) { |
| + frame2 = f; |
| + var channel = new MessageChannel(); |
| + channel.port1.onmessage = t.step_func(on_message); |
| + f.contentWindow.navigator.serviceWorker.controller.postMessage( |
| + {port:channel.port2}, [channel.port2]); |
| + }) |
| + .catch(unreached_rejection(t)); |
| + }, 'Client.id returns the client\'s UUID.'); |
| + |
| +// A regex object for UUID(http://tools.ietf.org/html/rfc4122) validation. |
| +var pattern = new RegExp(['^[0-9a-f]{8}-[0-9a-f]{4}-[1-5][0-9a-f]{3}-[89ab][0-', |
| + '9a-f]{3}-[0-9a-f]{12}$'].join(''), 'i'); |
| + |
| +function validate_uuid(id) { |
| + return id.match(pattern) ? true : false; |
|
falken
2015/06/26 02:21:51
This looks weird to me but I may be unfamiliar wit
jungkees
2015/06/26 02:43:22
Yes "return !!id.match(pattern);" makes sense for
|
| +} |
| + |
| +function on_message(e) { |
| + // The result of two sequential clients.matchAll() calls in the SW. |
| + // 1st matchAll() results in e.data[0], e.data[1]. |
| + // 2nd matchAll() results in e.data[2], e.data[3]. |
| + assert_equals(e.data.length, 4); |
| + // All should be valid UUIDs. |
| + assert_true(validate_uuid(e.data[0])); |
| + assert_true(validate_uuid(e.data[1])); |
| + assert_true(validate_uuid(e.data[2])); |
| + assert_true(validate_uuid(e.data[3])); |
| + // Different clients should have different ids. |
| + assert_not_equals(e.data[0], e.data[1]); |
| + assert_not_equals(e.data[2], e.data[3]); |
| + // Same clients should have an identical id. |
| + assert_equals(e.data[0], e.data[2]); |
| + assert_equals(e.data[1], e.data[3]); |
| + frame1.remove(); |
| + frame2.remove(); |
| + service_worker_unregister_and_done(test, scope); |
| +} |
| +</script> |
