third_party/WebKit/LayoutTests/http/tests/origin_trials/webexposed/budget-origin-trial-interfaces-in-service-worker.html - Issue 2812223004: Adding an origin trial for Budget API

Unified Diff: third_party/WebKit/LayoutTests/http/tests/origin_trials/webexposed/budget-origin-trial-interfaces-in-service-worker.html

Issue 2812223004: Adding an origin trial for Budget API (Closed)

Patch Set: Fix expected output for virtual LayoutTests Created 3 years, 8 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 side-by-side diff with in-line comments

« no previous file with comments | « third_party/WebKit/LayoutTests/http/tests/origin_trials/webexposed/budget-origin-trial-interfaces.html ('k') | third_party/WebKit/LayoutTests/virtual/origin-trials-runtimeflags-disabled/http/tests/origin_trials/webexposed/budget-origin-trial-interfaces-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: third_party/WebKit/LayoutTests/http/tests/origin_trials/webexposed/budget-origin-trial-interfaces-in-service-worker.html

diff --git a/third_party/WebKit/LayoutTests/http/tests/origin_trials/webexposed/budget-origin-trial-interfaces-in-service-worker.html b/third_party/WebKit/LayoutTests/http/tests/origin_trials/webexposed/budget-origin-trial-interfaces-in-service-worker.html

new file mode 100644

index 0000000000000000000000000000000000000000..c04fc35a0aa726fe5801bbc312e6d1e0c7754642

--- /dev/null

+++ b/third_party/WebKit/LayoutTests/http/tests/origin_trials/webexposed/budget-origin-trial-interfaces-in-service-worker.html

@@ -0,0 +1,40 @@

+<!DOCTYPE html>

+<html>

+ <head>

+ <!-- Generate token with the command:

+ generate_token.py http://127.0.0.1:8000 BudgetQuery --expire-timestamp=2000000000

+ -- -->

+ <meta http-equiv="origin-trial" content="AgFtR2Ps1Z9M/FW14Tgcwbajvq7kvzc/b1SPPSaaucG/P4ba6xC/69I9v8Pqx4wbsJINoqMabs9GE/LxOnPRfQIAAABTeyJvcmlnaW4iOiAiaHR0cDovLzEyNy4wLjAuMTo4MDAwIiwgImZlYXR1cmUiOiAiQnVkZ2V0UXVlcnkiLCAiZXhwaXJ5IjogMjAwMDAwMDAwMH0" />

Peter Beverloo 2017/04/24 14:24:01 Ian, how does this work? The Service Worker will b

iclelland 2017/04/24 14:45:33 If this test is passing, and it appears that the t

If this test is passing, and it appears that the trial is enabled when it should not be, I would suggest that we shouldn't be satisfied with the test coverage :) The tests in LayoutTests/http/tests/origin_trials will *always* see the budget feature available, because the runtime flag for experimental features is turned on, I believe. The tests there are useful for ensuring that the properties are bound correctly, because Chrome still uses the origin trials mechanism to attach them to the JS objects. Tests in LayoutTests/virtual/origin-trials-runtimeflags-disabled/ run with the runtime flag disabled, but you should still be able to turn the feature on with a trial token. The service worker tests should pass in this directory if we add the token as an HTTP header (see https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/http/test... as an example)

Origin trials propagate to dedicated worker threads from the document which creates them, but not to shared workers or service workers. Those need to have the trial enabled by an HTTP header (we could probably add a JS interface, but have not done that yet for worker scripts). The token is cached with the service worker version, so that if the service worker is restarted without a network request, the token will still be enabled.

In both cases, the presence of the token in the HTTP response which delivers the worker script will determine whether the trial is enabled in that context.

+ <title>Budget interfaces available in a Service Worker</title>

+ <script src="../../resources/testharness.js"></script>

+ <script src="../../resources/testharnessreport.js"></script>

+ <script src="../../serviceworker/resources/test-helpers.js"></script>

+ <script src="../../notifications/resources/test-helpers.js"></script>

+ </head>

+ <body>

+ <script>

+ promise_test(function(test) {

+ const script = '../../budget/resources/instrumentation-service-worker.js';

+ const scope = '../../budget/resources/scope/' + location.pathname;

+ let port;

Peter Beverloo 2017/04/24 14:24:01 nit: |port| can be local to the function on line 2

+ return getActiveServiceWorkerWithMessagePort(test, script, scope)

+ .then(function(workerInfo) {

+ port = workerInfo.port;

+ port.addEventListener('message', function(event) {

+ if (typeof event.data != 'object' || !event.data.command)

+ assert_unreached('Invalid message from the service worker');

+ assert_equals(event.data.command, 'checkInterfaces');

+ assert_true(event.data.success,

+ 'checkInterfaces should succeed but failed with error: ' +

+ event.data.message);

+ });

+ return sendCommand(port, { command: 'checkInterfaces' });

Peter Beverloo 2017/04/24 14:24:01 Are we sure that the event listener on line 25 is

+ })

+ }, 'BudgetQuery interfaces should be available in a Service Worker.');

+ </script>

+ </body>

+</html>