Merge pull request #15 from PerfLab-io/export-trace

Add event listener to export trace data using TimelinePanel.saveFile

Author: WebCloud

front_end/entrypoints/main/MainImpl.ts

@@ -769,6 +769,7 @@ export class MainImpl {
     // this.#container.addEventListener('contextmenu', this.#contextMenuEventFired.bind(this), true);
     this.#container.addEventListener(Timeline.TimelinePanel.Events.OpenTraceFile, Timeline.TimelinePanel.TimelinePanel.instance().selectFileToLoad.bind(Timeline.TimelinePanel.TimelinePanel.instance()));
     this.#container.addEventListener(Timeline.TimelinePanel.Events.LoadRawTraceData, Timeline.TimelinePanel.TimelinePanel.instance().getRawTraceData.bind(Timeline.TimelinePanel.TimelinePanel.instance()));
+    this.#container.addEventListener(Timeline.TimelinePanel.Events.ExportTrace, Timeline.TimelinePanel.TimelinePanel.instance().exportTrace.bind(Timeline.TimelinePanel.TimelinePanel.instance()));
     // @ts-ignore
     this.#container.addEventListener(Timeline.TimelinePanel.Events.LoadTraceFile, Timeline.TimelinePanel.TimelinePanel.instance().loadTraceFile.bind(Timeline.TimelinePanel.TimelinePanel.instance()));
   }

front_end/panels/timeline/TimelinePanel.ts

@@ -1076,7 +1076,11 @@ export class TimelinePanel extends UI.Panel.Panel implements Client, TimelineMod
     }
   }
 
-  async saveToFile(isEnhancedTraces: boolean = false, addModifications: boolean = false): Promise<void> {
+  async exportTrace(): Promise<void> {
+    void this.saveToFile();
+  }
+
+  async saveToFile(isEnhancedTraces: boolean = false, addModifications: boolean = true): Promise<void> {
     if (this.state !== State.IDLE) {
       return;
     }
@@ -2251,13 +2255,7 @@ export const enum Events {
   RawTraceDataLoaded = 'rawtracedataloaded',
   LoadTraceFile = 'loadtracefile',
   TraceLoadingStarted = 'traceloadingstarted',
-}
-
-export class OpenTraceFileEvent extends CustomEvent<Events.OpenTraceFile> {
-  static readonly eventName = Events.OpenTraceFile;
-  constructor() {
-    super(Events.OpenTraceFile);
-  }
+  ExportTrace = 'exporttrace',
 }
 
 export type EventTypes = {
@@ -2269,6 +2267,20 @@ export type EventTypes = {
   },
 };
 
+export class OpenTraceFileEvent extends CustomEvent<Events.OpenTraceFile> {
+  static readonly eventName = Events.OpenTraceFile;
+  constructor() {
+    super(Events.OpenTraceFile);
+  }
+}
+
+export class ExportTraceEvent extends CustomEvent<Events.ExportTrace> {
+  static readonly eventName = Events.ExportTrace;
+  constructor() {
+    super(Events.ExportTrace);
+  }
+}
+
 export class LoadTraceFileEvent extends CustomEvent<EventTypes[Events.LoadTraceFile]> {
   static readonly eventName = Events.LoadTraceFile;
   constructor(options: EventTypes[Events.LoadTraceFile]) {
@@ -2510,15 +2522,15 @@ export class ActionDelegate implements UI.ActionRegistration.ActionDelegate {
       case 'timeline.jump-to-next-frame':
         panel.jumpToFrame(1);
         return true;
-      case 'timeline.show-history':
-        void panel.showHistoryDropdown();
-        return true;
-      case 'timeline.previous-recording':
-        panel.navigateHistory(1);
-        return true;
-      case 'timeline.next-recording':
-        panel.navigateHistory(-1);
-        return true;
+      // case 'timeline.show-history':
+      //   void panel.showHistoryDropdown();
+      //   return true;
+      // case 'timeline.previous-recording':
+      //   panel.navigateHistory(1);
+      //   return true;
+      // case 'timeline.next-recording':
+      //   panel.navigateHistory(-1);
+      //   return true;
     }
     return false;
   }