Skip to content

GitLab

T
tangled
Commit 8556c639 authored by Yutaka Hirano's avatar Yutaka Hirano Committed by Commit Bot
Browse files
Options

Remove http/tests/reporting-observer/reporting-observer-worker.html 

We have used deprecation reports for Atomics.wake to test
ReportingObserver for workers, but now Atomics.waks is being removed so
this test will not work.

We have some WPTs using ReportingObservers in workers, in
wpt/html/cross-origin-embedder-policy, so we won't lose test coverage
by removing this test.

Bug: v8:7883
Change-Id: I63728234f0c008d826cb8c889461a0ec8b17ce53
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2336131
Auto-Submit: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Ian Clelland <iclelland@chromium.org>
Reviewed-by: default avatarIan Clelland <iclelland@chromium.org>
Cr-Commit-Position: refs/heads/master@{#796415}
parent 0487b79f
Hide whitespace changes
Inline Side-by-side
Showing with 0 additions and 95 deletions
third_party/blink/web_tests/http/tests/reporting-observer/reporting-observer-worker.html deleted 100644 → 0
View file @ 0487b79f
<!doctype html>
<html>
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
<script src="/serviceworker/resources/test-helpers.js"></script>
<script>
const WORKER_URL
= new URL('resources/deprecation-worker.js', location).href;
function checkReport(report) {
assert_equals(report.type, 'deprecation');
assert_equals(report.url, WORKER_URL);
assert_equals(typeof report.body.id, 'string');
assert_equals(typeof report.body.anticipatedRemoval, 'string');
assert_equals(typeof report.body.message, 'string');
assert_equals(report.body.sourceFile, WORKER_URL);
assert_equals(typeof report.body.lineNumber, 'number');
assert_equals(typeof report.body.columnNumber, 'number');
}
promise_test(async (test) => {
const worker = new Worker(WORKER_URL);
test.add_cleanup(() => worker.terminate());
const mc = new MessageChannel();
worker.postMessage(mc.port2, [mc.port2]);
test.add_cleanup(() => mc.port1.close());
const reports = (await new Promise(r => {
mc.port1.onmessage = r;
})).data;
assert_equals(reports.length, 1);
checkReport(reports[0]);
}, 'Deprecation reports on DedicatedWorker');
promise_test(async (test) => {
const worker = new SharedWorker(WORKER_URL);
const mc = new MessageChannel();
worker.port.start();
worker.port.postMessage(mc.port2, [mc.port2]);
test.add_cleanup(() => mc.port1.close());
const reports = (await new Promise(r => {
mc.port1.onmessage = r;
})).data;
assert_equals(reports.length, 1);
checkReport(reports[0]);
}, 'Deprecation reports on SharedWorker');
promise_test(async (test) => {
const SCOPE = new URL('resources/empty.html', location).pathname;
const reg = await service_worker_unregister_and_register(test, WORKER_URL, SCOPE);
await wait_for_state(test, reg.installing, 'activated');
test.add_cleanup(() => reg.unregister());
const frame = await with_iframe(SCOPE);
const mc = new MessageChannel();
const worker = frame.contentWindow.navigator.serviceWorker.controller;
worker.postMessage(mc.port2, [mc.port2]);
test.add_cleanup(() => mc.port1.close());
const reports = (await new Promise(r => {
mc.port1.onmessage = r;
})).data;
assert_equals(reports.length, 1);
checkReport(reports[0]);
}, 'Deprecation reports on ServiceWorker');
</script>
</html>
\ No newline at end of file
third_party/blink/web_tests/http/tests/reporting-observer/resources/deprecation-worker.js deleted 100644 → 0
View file @ 0487b79f
function run(port) {
const observer = new ReportingObserver((reports) => {
port.postMessage(reports.map(report => report.toJSON()));
});
observer.observe();
try {
// This API is deprecated. This throws an exception because of a bad
// argument, but we don't care.
Atomics.wake();
} catch (e) {
}
}
// For DedicatedWorker and ServiceWorker
self.addEventListener('message', (e) => run(e.data));
// For SharedWorker
self.addEventListener('connect', (e) => {
e.ports[0].onmessage = (ev) => run(ev.data);
});
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment