Remove _processSourceTable and only add the sources while we are processing the JS functions

Author: canova

src/profile-logic/global-data-collector.ts

@@ -83,11 +83,6 @@ export class GlobalDataCollector {
     return index;
   }
 
-  // Get the processed source index by UUID
-  getSourceIndexByUuid(uuid: string): IndexIntoSourceTable | null {
-    return this._uuidToSourceIndex.get(uuid) ?? null;
-  }
-
   getStringTable(): StringTable {
     return this._stringTable;
   }

src/profile-logic/process-profile.ts

@@ -536,8 +536,10 @@ function _extractJsFunction(
     // Look up the UUID for this source index from the process's sources table
     if (geckoSourceTable && geckoSourceIdx < geckoSourceTable.data.length) {
       const uuidIndex = geckoSourceTable.schema.uuid;
+      const filenameIndex = geckoSourceTable.schema.filename;
       const uuid = geckoSourceTable.data[geckoSourceIdx][uuidIndex];
-      processedSourceIndex = globalDataCollector.getSourceIndexByUuid(uuid);
+      const filename = geckoSourceTable.data[geckoSourceIdx][filenameIndex];
+      processedSourceIndex = globalDataCollector.indexForSource(uuid, filename);
     }
   }
 
@@ -1657,37 +1659,6 @@ export function processGeckoOrDevToolsProfile(json: unknown): Profile {
   return processGeckoProfile(geckoProfile);
 }
 
-/**
- * Process source table from all processes and populate the global data
- * collector with UUID-to-index mappings.
- */
-function _processSourceTable(
-  geckoProfile: GeckoProfile,
-  globalDataCollector: GlobalDataCollector
-): void {
-  // Process the main process sources table if it exists
-  if (geckoProfile.sources) {
-    const schema = geckoProfile.sources.schema;
-    for (const data of geckoProfile.sources.data) {
-      const uuid = data[schema.uuid];
-      const filename = data[schema.filename];
-      globalDataCollector.indexForSource(uuid, filename);
-    }
-  }
-
-  // Process sources from all subprocesses
-  for (const subprocessProfile of geckoProfile.processes) {
-    if (subprocessProfile.sources) {
-      const schema = subprocessProfile.sources.schema;
-      for (const data of subprocessProfile.sources.data) {
-        const uuid = data[schema.uuid];
-        const filename = data[schema.filename];
-        globalDataCollector.indexForSource(uuid, filename);
-      }
-    }
-  }
-}
-
 /**
  * Convert a profile from the Gecko format into the processed format.
  * Throws an exception if it encounters an incompatible profile.
@@ -1710,9 +1681,6 @@ export function processGeckoProfile(geckoProfile: GeckoProfile): Profile {
 
   const globalDataCollector = new GlobalDataCollector();
 
-  // Process sources from all processes (main + subprocesses)
-  _processSourceTable(geckoProfile, globalDataCollector);
-
   for (const thread of geckoProfile.threads) {
     threads.push(
       _processThread(