Skip to content

Commit 78ac9e2

Browse files
Lightning00BladeDevtools-frontend LUCI CQ
authored andcommitted
Fix CXX extension build
Updated to the helpers made this fail. Fixed: 474387339 Change-Id: Icf72f4f048a087800a5f2f9b8bf259dc444fb1f4 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/7414707 Auto-Submit: Nikolay Vitkov <[email protected]> Commit-Queue: Samiya Caur <[email protected]> Commit-Queue: Nikolay Vitkov <[email protected]> Reviewed-by: Samiya Caur <[email protected]>
1 parent 64070c6 commit 78ac9e2

File tree

3 files changed

+11
-11
lines changed

3 files changed

+11
-11
lines changed

extensions/cxx_debugging/e2e/TestDriver.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ describe.skip('[crbug.com/468345402] CXX Debugging Extension Test Suite', functi
8484

8585
assert.strictEqual(stoppedText, pausedReasonText(reason));
8686

87-
const pausedLocation = await retrieveTopCallFrameWithoutResuming();
87+
const pausedLocation = await retrieveTopCallFrameWithoutResuming(devToolsPage);
8888
if (pausedLocation?.includes('…')) {
8989
const pausedLocationSplit = pausedLocation.split('…');
9090
assert.isTrue(
@@ -123,10 +123,10 @@ describe.skip('[crbug.com/468345402] CXX Debugging Extension Test Suite', functi
123123
// TODO(jarin) Without waiting here, the FE often misses the click on the console tab.
124124
await devToolsPage.timeout(500);
125125
await devToolsPage.click(CONSOLE_TAB_SELECTOR);
126-
await focusConsolePrompt();
126+
await focusConsolePrompt(devToolsPage);
127127

128128
for (const {expression, value} of evaluations) {
129-
await typeIntoConsoleAndWaitForResult(expression);
129+
await typeIntoConsoleAndWaitForResult(expression, undefined, undefined, devToolsPage);
130130
const evaluateResults = await devToolsPage.evaluate(() => {
131131
return Array.from(document.querySelectorAll('.console-user-command-result'))
132132
.map(node => node.textContent);
@@ -135,7 +135,7 @@ describe.skip('[crbug.com/468345402] CXX Debugging Extension Test Suite', functi
135135
assert.strictEqual(result, value.toString());
136136
}
137137

138-
await openSourcesPanel();
138+
await openSourcesPanel(devToolsPage);
139139
}
140140

141141
if (thread) {
@@ -250,9 +250,9 @@ async function doActions({actions, reason}: {actions?: Action[], reason: string}
250250
if (!breakpoint) {
251251
throw new Error('Invalid breakpoint spec: missing `breakpoint`');
252252
}
253-
await openFileInEditor(file);
253+
await openFileInEditor(file, devToolsPage);
254254
await scrollToLine(Number(breakpoint));
255-
await addBreakpointForLine(breakpoint);
255+
await addBreakpointForLine(breakpoint, devToolsPage);
256256
break;
257257
}
258258
case 'remove_breakpoint': {
@@ -261,7 +261,7 @@ async function doActions({actions, reason}: {actions?: Action[], reason: string}
261261
throw new Error('Invalid breakpoint spec: missing `breakpoint`');
262262
}
263263
await scrollToLine(Number(breakpoint));
264-
await removeBreakpointForLine(breakpoint);
264+
await removeBreakpointForLine(breakpoint, devToolsPage);
265265
break;
266266
}
267267
case 'step_over':

extensions/cxx_debugging/e2e/cxx-debugging-extension-helpers.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,10 +45,10 @@ export interface TestSpec {
4545
}
4646

4747
export async function openTestSuiteResourceInSourcesPanel(testInput: string) {
48-
const {inspectedPage} = getBrowserAndPagesWrappers();
48+
const {inspectedPage, devToolsPage} = getBrowserAndPagesWrappers();
4949
await inspectedPage.goTo(`${getTestsuiteResourcesPath()}/extension_test_suite/${testInput}`);
5050

51-
await openSourcesPanel();
51+
await openSourcesPanel(devToolsPage);
5252
}
5353

5454
export function getTestsuiteResourcesPath() {

extensions/cxx_debugging/e2e/standalone/MemoryInspector.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,8 @@ describe('LinearMemoryInspector', () => {
2424

2525
const file = 'scope-view-primitives.c';
2626
const breakpoint = 14;
27-
await openFileInEditor(file);
28-
await addBreakpointForLine(Number(breakpoint));
27+
await openFileInEditor(file, devToolsPage);
28+
await addBreakpointForLine(Number(breakpoint), devToolsPage);
2929

3030
await inspectedPage.reload();
3131
await devToolsPage.waitForFunction(

0 commit comments

Comments
 (0)