Commit bbf439e4 authored by Andrey Kosyakov's avatar Andrey Kosyakov Committed by Chromium LUCI CQ

DevTools: prepare tests for introduction of ExecutionContextDescription.uniqueID

This temporarily removes the newly added field from objects being dumped,
so that the test expectations do not change when upstream CL lands.

Bug: v8:11268, chromium:1101897
Change-Id: I8caa2bcf4a5b71250fb0712129771215c3a215bb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2601547Reviewed-by: default avatarPeter Kvitek <kvitekp@chromium.org>
Commit-Queue: Andrey Kosyakov <caseq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#839032}
parent 2eaf311e
......@@ -3,6 +3,8 @@
`Tests that execution contexts are reported for iframes that don't have src attribute.`);
await page.navigate('http://devtools.test:8000/inspector-protocol/resources/iframe-no-src.html');
dp.Runtime.onExecutionContextCreated(event => {
// TODO(caseq): remove following v8 roll past https://chromium-review.googlesource.com/c/v8/v8/+/2594538
delete event.params.context.uniqueId;
testRunner.log(event);
});
await dp.Runtime.enable();
......
......@@ -4,6 +4,9 @@
await dp.Runtime.enable();
let count = 0;
dp.Runtime.onExecutionContextCreated(event => {
// TODO(caseq): remove following v8 roll past https://chromium-review.googlesource.com/c/v8/v8/+/2594538
delete event.params.context.uniqueId;
testRunner.log(event);
if (++count === 2) // page context + frame context.
testRunner.completeTest();
......
......@@ -3,6 +3,9 @@
`Tests that execution contexts are reported for frames that were blocked due to mixed content when Runtime is enabled *after* navigation.`);
await page.navigate('https://devtools.test:8443/inspector-protocol/resources/mixed-content-iframe.html');
dp.Runtime.onExecutionContextCreated(event => {
// TODO(caseq): remove following v8 roll past https://chromium-review.googlesource.com/c/v8/v8/+/2594538
delete event.params.context.uniqueId;
testRunner.log(event);
});
await dp.Runtime.enable();
......
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