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

Side by Side Diff: LayoutTests/http/tests/serviceworker/chromium/clients-openwindow.html

Issue 985633003: Service Worker: Clients.openWindow() should allow opening x-origin URLs (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 years, 9 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 unified diff | Download patch
« no previous file with comments | « no previous file | LayoutTests/http/tests/serviceworker/chromium/resources/clients-openwindow.js » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!DOCTYPE html> 1 <!DOCTYPE html>
2 <title>Service Worker: clients.openWindow() tests (using testRunner)</title> 2 <title>Service Worker: clients.openWindow() tests (using testRunner)</title>
3 <script src="../../resources/testharness.js"></script> 3 <script src="../../resources/testharness.js"></script>
4 <script src="../../resources/testharnessreport.js"></script> 4 <script src="../../resources/testharnessreport.js"></script>
5 <script src="../resources/test-helpers.js"></script> 5 <script src="../resources/test-helpers.js"></script>
6 <script> 6 <script>
7 // This test is using testRunner to grant itself the notification permission and 7 // This test is using testRunner to grant itself the notification permission and
8 // to simulate a click on a notification. A couple of changes would allow it to 8 // to simulate a click on a notification. A couple of changes would allow it to
9 // be run as a manual test by other browser vendors. 9 // be run as a manual test by other browser vendors.
10 if (window.testRunner) 10 if (window.testRunner)
(...skipping 11 matching lines...) Expand all
22 .then(function(frame) { 22 .then(function(frame) {
23 var w = frame.contentWindow; 23 var w = frame.contentWindow;
24 w.onmessage = t.step_func(onMessage); 24 w.onmessage = t.step_func(onMessage);
25 w.navigator.serviceWorker.controller.postMessage('start'); 25 w.navigator.serviceWorker.controller.postMessage('start');
26 }) 26 })
27 .catch(unreached_rejection(t)); 27 .catch(unreached_rejection(t));
28 28
29 var result = []; 29 var result = [];
30 var expected = ['openWindow() can\'t open a window without a user interactio n', 30 var expected = ['openWindow() can\'t open a window without a user interactio n',
31 'openWindow() error is InvalidAccessError', 31 'openWindow() error is InvalidAccessError',
32 'openWindow() can\'t open cross origin windows', 32 'openWindow() can open cross origin windows',
33 'openWindow() error is SecurityError', 33 'openWindow() result: null',
34 'openWindow() can open not controlled windows', 34 'openWindow() can open not controlled windows',
35 'openWindow() result: null', 35 'openWindow() result: null',
36 'openWindow() can open controlled windows', 36 'openWindow() can open controlled windows',
37 'openWindow() result: [object WindowClient]', 37 'openWindow() result: [object WindowClient]',
38 ' url: ' + location.origin + '/serviceworker/chromium/resour ces/blank.html', 38 ' url: ' + location.origin + '/serviceworker/chromium/resour ces/blank.html',
39 ' visibilityState: visible', 39 ' visibilityState: visible',
40 ' focused: true', 40 ' focused: true',
41 ' frameType: top-level']; 41 ' frameType: top-level'];
42 42
43 // LayoutTests on Mac do not open focused windows. 43 // LayoutTests on Mac do not open focused windows.
(...skipping 15 matching lines...) Expand all
59 if (message === 'quit') { 59 if (message === 'quit') {
60 assert_array_equals(result, expected, 60 assert_array_equals(result, expected,
61 'Worker should post back expected messages.'); 61 'Worker should post back expected messages.');
62 service_worker_unregister_and_done(t, scope); 62 service_worker_unregister_and_done(t, scope);
63 } else { 63 } else {
64 result.push(message); 64 result.push(message);
65 } 65 }
66 } 66 }
67 }); 67 });
68 </script> 68 </script>
OLDNEW
« no previous file with comments | « no previous file | LayoutTests/http/tests/serviceworker/chromium/resources/clients-openwindow.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698