Skip to content

Commit c71cecc

Browse files
jackfranklinDevtools-frontend LUCI CQ
authored andcommitted
Insights: add interaction + screenshot test
Bug: 373355645 Change-Id: I7bdaf63cc6d33341ad2781b5ec9a8792466350aa Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/5928967 Auto-Submit: Jack Franklin <[email protected]> Commit-Queue: Jack Franklin <[email protected]> Reviewed-by: Alina Varkki <[email protected]>
1 parent 546c5dd commit c71cecc

File tree

16 files changed

+69
-5
lines changed

16 files changed

+69
-5
lines changed

front_end/panels/timeline/TimelinePanel.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -279,9 +279,9 @@ const UIStrings = {
279279
*/
280280
showSidebar: 'Show sidebar',
281281
/**
282-
* @description Tooltip for the the sole sidebar toggle in the Performance panel. Command to close the sidebar.
282+
* @description Tooltip for the the sidebar toggle in the Performance panel. Command to close the sidebar.
283283
*/
284-
hideSidebar: 'Hide sole sidebar',
284+
hideSidebar: 'Hide sidebar',
285285
/**
286286
* @description Screen reader announcement when the sidebar is shown in the Performance panel.
287287
*/
@@ -1831,6 +1831,10 @@ export class TimelinePanel extends UI.Panel.Panel implements Client, TimelineMod
18311831
* We also check that the experiments are enabled, else we reveal an entirely empty sidebar.
18321832
*/
18331833
#showSidebarIfRequired(): void {
1834+
if (Root.Runtime.Runtime.queryParam('disable-auto-performance-sidebar-reveal') !== null) {
1835+
// Used in interaction tests & screenshot tests.
1836+
return;
1837+
}
18341838
const needToRestore = this.#restoreSidebarVisibilityOnTraceLoad;
18351839
const userHasSeenSidebar = this.#sideBar.userHasOpenedSidebarOnce();
18361840
const experimentsEnabled = this.#panelSidebarEnabled();

front_end/panels/timeline/components/Sidebar.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ export class SidebarWidget extends UI.Widget.VBox {
6565
* SplitWidget tracks its state in its own setting.
6666
*/
6767
#userHasOpenedSidebarOnce =
68-
Common.Settings.Settings.instance().createSetting<boolean>('timeline-user-has-opened-siderbar-once', false);
68+
Common.Settings.Settings.instance().createSetting<boolean>('timeline-user-has-opened-sidebar-once', false);
6969

7070
userHasOpenedSidebarOnce(): boolean {
7171
return this.#userHasOpenedSidebarOnce.get();

front_end/ui/components/docs/performance_panel/basic.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,10 @@ const cpuprofileName = params.get('cpuprofile');
101101
const traceUrl = params.get('loadTimelineFromURL');
102102
const nodeMode = params.get('isNode');
103103
const isNodeMode = nodeMode === 'true' ? true : false;
104-
Root.Runtime.experiments.setEnabled('timeline-invalidation-tracking', params.has('invalidations'));
104+
105+
// These are both enabled by default in Chrome M131 and will be removed in M132.
106+
Root.Runtime.experiments.setEnabled(Root.Runtime.ExperimentName.TIMELINE_INSIGHTS, true);
107+
Root.Runtime.experiments.setEnabled(Root.Runtime.ExperimentName.TIMELINE_ANNOTATIONS, true);
105108

106109
const timeline = Timeline.TimelinePanel.TimelinePanel.instance({forceNew: true, isNode: isNodeMode});
107110
const container = document.getElementById('container');

front_end/ui/visual_logging/KnownContextValues.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3002,6 +3002,7 @@ export const knownContextValues = new Set([
30023002
'timeline-status',
30033003
'timeline-tree-current-thread',
30043004
'timeline-tree-group-by',
3005+
'timeline-user-has-opened-sidebar-once',
30053006
'timeline-user-has-opened-siderbar-once',
30063007
'timeline-v8-runtime-call-stats',
30073008
'timeline.animations',
-347 Bytes
Loading
-327 Bytes
Loading
38 KB
Loading
38 KB
Loading
37.9 KB
Loading
110 KB
Loading

0 commit comments

Comments
 (0)