Skip to content

Commit db4ee92

Browse files
szuendDevtools-frontend LUCI CQ
authored andcommitted
[ui] Use 'StackTracePreview' widget instead of plain function
Migrates two more use sites. [email protected] Bug: 407752215,433162438 Change-Id: Iacca303c0939fcabc7c8bf005f42e03bd61dabfb Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6796895 Commit-Queue: Simon Zünd <[email protected]> Reviewed-by: Danil Somsikov <[email protected]>
1 parent 2688ed2 commit db4ee92

File tree

4 files changed

+12
-9
lines changed

4 files changed

+12
-9
lines changed

front_end/panels/console/ConsoleViewMessage.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ describeWithMockConnection('ConsoleViewMessage', () => {
172172
const IGNORE_LIST_LINK = 'ignore-list-link';
173173

174174
function findStackPreviewContainer(element: HTMLElement) {
175-
const outer = element.querySelector('span.stack-preview-container');
175+
const outer = element.querySelector('.stack-preview-container');
176176
assert.isNotNull(outer);
177177
const inner = outer.shadowRoot;
178178
assert.isNotNull(inner);

front_end/panels/console/ConsoleViewMessage.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -705,10 +705,11 @@ export class ConsoleViewMessage implements ConsoleViewportElement {
705705
clickableElement.tabIndex = -1;
706706
clickableElement.appendChild(messageElement);
707707
const stackTraceElement = contentElement.createChild('div');
708-
const stackTracePreview = Components.JSPresentationUtils.buildStackTracePreviewContents(
709-
target, this.linkifier, {stackTrace, tabStops: undefined, widthConstrained: true});
710-
stackTraceElement.appendChild(stackTracePreview.element);
711-
for (const linkElement of stackTracePreview.links) {
708+
const stackTracePreview = new Components.JSPresentationUtils.StackTracePreviewContent(
709+
undefined, target ?? undefined, this.linkifier, {stackTrace, widthConstrained: true});
710+
stackTracePreview.markAsRoot();
711+
stackTracePreview.show(stackTraceElement);
712+
for (const linkElement of stackTracePreview.linkElements) {
712713
this.selectableChildren.push({element: linkElement, forceSelect: () => linkElement.focus()});
713714
}
714715
stackTraceElement.classList.add('hidden-stack-trace');

front_end/panels/timeline/TimelineUIUtils.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2407,9 +2407,11 @@ export class TimelineDetailsContentHelper {
24072407
}
24082408
const stackTraceElement =
24092409
this.tableElement.createChild('div', 'timeline-details-view-row timeline-details-stack-values');
2410-
const callFrameContents = LegacyComponents.JSPresentationUtils.buildStackTracePreviewContents(
2411-
this.target, this.linkifierInternal, {stackTrace: resolvedStackTrace, tabStops: true, showColumnNumber: true});
2412-
stackTraceElement.appendChild(callFrameContents.element);
2410+
const callFrameContents = new LegacyComponents.JSPresentationUtils.StackTracePreviewContent(
2411+
undefined, this.target ?? undefined, this.linkifierInternal,
2412+
{stackTrace: resolvedStackTrace, tabStops: true, showColumnNumber: true});
2413+
callFrameContents.markAsRoot();
2414+
callFrameContents.show(stackTraceElement);
24132415
}
24142416
}
24152417

test/e2e/helpers/console-helpers.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,7 @@ export async function getStructuredConsoleMessages(devToolsPage?: DevToolsPage)
228228
const source = wrapper.querySelector('.devtools-link')?.textContent;
229229
const consoleMessage = wrapper.querySelector('.console-message');
230230
const repeatCount = wrapper.querySelector('.console-message-repeat-count');
231-
const stackPreviewRoot = wrapper.querySelector('.hidden-stack-trace > span');
231+
const stackPreviewRoot = wrapper.querySelector('.hidden-stack-trace > div');
232232
const stackPreview = stackPreviewRoot?.shadowRoot?.querySelectorAll('tbody') ?? null;
233233
return {
234234
message,

0 commit comments

Comments
 (0)