Skip to content

Commit 41d973f

Browse files
Lightning00BladeDevtools-frontend LUCI CQ
authored andcommitted
[e2e] Don't show Node icon under tests
Bug: none Change-Id: I2c88a032d22d7fd3337a99fa635b750930d8daf7 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6613142 Auto-Submit: Nikolay Vitkov <nvitkov@chromium.org> Reviewed-by: Alex Rudenko <alexrudenko@chromium.org> Commit-Queue: Alex Rudenko <alexrudenko@chromium.org>
1 parent 1911c47 commit 41d973f

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

front_end/entrypoints/inspector_main/InspectorMain.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -215,6 +215,11 @@ export class NodeIndicator implements UI.Toolbar.Provider {
215215
}
216216

217217
#update(targetInfos: Protocol.Target.TargetInfo[]): void {
218+
// Disable when we are testing, as debugging e2e
219+
// attaches a debug process and this changes some view sizes
220+
if (Host.InspectorFrontendHost.isUnderTest()) {
221+
return;
222+
}
218223
const hasNode = Boolean(targetInfos.find(target => target.type === 'node' && !target.attached));
219224
this.#element.classList.toggle('inactive', !hasNode);
220225
if (hasNode) {

0 commit comments

Comments
 (0)