skip reading element for imported data

facebook · May 13, 2020 · 30a55f0 · 30a55f0
1 parent 6514e4a
commit 30a55f0
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 10 deletions.
diff --git a/packages/react-devtools-shared/src/devtools/store.js b/packages/react-devtools-shared/src/devtools/store.js
@@ -416,6 +416,11 @@ export default class Store extends EventEmitter<{|
   }
 
   getElementByID(id: number): Element | null {
+    if (
+      this._profilerStore._dataFrontend &&
+      this._profilerStore._dataFrontend.imported === true
+    )
+      return null;
     const element = this._idToElement.get(id);
     if (element == null) {
       console.warn(`No element found with id "${id}"`);

diff --git a/packages/react-devtools-shared/src/devtools/views/Profiler/ProfilingImportExportButtons.js b/packages/react-devtools-shared/src/devtools/views/Profiler/ProfilingImportExportButtons.js
@@ -16,7 +16,7 @@ import ButtonIcon from '../ButtonIcon';
 import {StoreContext} from '../context';
 import {
   prepareProfilingDataExport,
-  prepareProfilingDataFrontendFromExport,
+  prepareProfilingDataFrontendFromImport,
 } from './utils';
 import {downloadFile} from '../utils';
 
@@ -77,11 +77,11 @@ export default function ProfilingImportExportButtons() {
       fileReader.addEventListener('load', () => {
         try {
           const raw = ((fileReader.result: any): string);
-          const profilingDataExport = ((JSON.parse(
+          const profilingDataImport = ((JSON.parse(
             raw,
           ): any): ProfilingDataExport);
-          profilerStore.profilingData = prepareProfilingDataFrontendFromExport(
-            profilingDataExport,
+          profilerStore.profilingData = prepareProfilingDataFrontendFromImport(
+            profilingDataImport,
           );
         } catch (error) {
           modalDialogDispatch({

diff --git a/packages/react-devtools-shared/src/devtools/views/Profiler/types.js b/packages/react-devtools-shared/src/devtools/views/Profiler/types.js
@@ -105,6 +105,7 @@ export type ProfilingDataForRootFrontend = {|
 export type ProfilingDataFrontend = {|
   // Profiling data per root.
   dataForRoots: Map<number, ProfilingDataForRootFrontend>,
+  imported: boolean,
 |};
 
 export type CommitDataExport = {|

diff --git a/packages/react-devtools-shared/src/devtools/views/Profiler/utils.js b/packages/react-devtools-shared/src/devtools/views/Profiler/utils.js
@@ -99,21 +99,21 @@ export function prepareProfilingDataFrontendFromBackendAndStore(
     );
   });
 
-  return {dataForRoots};
+  return {dataForRoots, imported: false};
 }
 
 // Converts a Profiling data export into the format required by the Store.
-export function prepareProfilingDataFrontendFromExport(
-  profilingDataExport: ProfilingDataExport,
+export function prepareProfilingDataFrontendFromImport(
+  profilingDataImport: ProfilingDataExport,
 ): ProfilingDataFrontend {
-  const {version} = profilingDataExport;
+  const {version} = profilingDataImport;
 
   if (version !== PROFILER_EXPORT_VERSION) {
     throw Error(`Unsupported profiler export version "${version}"`);
   }
 
   const dataForRoots: Map<number, ProfilingDataForRootFrontend> = new Map();
-  profilingDataExport.dataForRoots.forEach(
+  profilingDataImport.dataForRoots.forEach(
     ({
       commitData,
       displayName,
@@ -156,7 +156,7 @@ export function prepareProfilingDataFrontendFromExport(
     },
   );
 
-  return {dataForRoots};
+  return {dataForRoots, imported: true};
 }
 
 // Converts a Store Profiling data into a format that can be safely (JSON) serialized for export.