Bug 1529680 - [release 127] [Breakpoints] Improve column breakpoint performance (#8000). r=dwalsh
authorJason Laster <jason.laster.11@gmail.com>
Mon, 25 Feb 2019 10:02:35 -0500
changeset 518969 4f8f76780ecab209576d28700d0813622f96c2e2
parent 518968 065b2e8fef2bd8d77a18e41ddc6ca02402278f7a
child 518970 aadfff067f34a785196322e908c70d254b32698e
push id10862
push userffxbld-merge
push dateMon, 11 Mar 2019 13:01:11 +0000
treeherdermozilla-beta@a2e7f5c935da [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdwalsh
bugs1529680
milestone67.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1529680 - [release 127] [Breakpoints] Improve column breakpoint performance (#8000). r=dwalsh
devtools/client/debugger/new/packages/devtools-source-map/src/index.js
devtools/client/shared/source-map/index.js
--- a/devtools/client/debugger/new/packages/devtools-source-map/src/index.js
+++ b/devtools/client/debugger/new/packages/devtools-source-map/src/index.js
@@ -20,16 +20,20 @@ const _getGeneratedRanges = dispatcher.t
 
 const _getGeneratedLocation = dispatcher.task("getGeneratedLocation", {
   queue: true
 });
 const _getAllGeneratedLocations = dispatcher.task("getAllGeneratedLocations", {
   queue: true
 });
 
+const _getOriginalLocation = dispatcher.task("getOriginalLocation", {
+  queue: true
+});
+
 export const setAssetRootURL = async (assetRoot: string): Promise<void> =>
   dispatcher.invoke("setAssetRootURL", assetRoot);
 
 export const getOriginalURLs = async (
   generatedSource: Source
 ): Promise<SourceMapConsumer> =>
   dispatcher.invoke("getOriginalURLs", generatedSource);
 
@@ -66,18 +70,17 @@ export const getAllGeneratedLocations = 
   location: SourceLocation,
   originalSource: Source
 ): Promise<Array<SourceLocation>> =>
   _getAllGeneratedLocations(location, originalSource);
 
 export const getOriginalLocation = async (
   location: SourceLocation,
   options: locationOptions = {}
-): Promise<SourceLocation> =>
-  dispatcher.invoke("getOriginalLocation", location, options);
+): Promise<SourceLocation> => _getOriginalLocation(location, options);
 
 export const getFileGeneratedRange = async (
   originalSource: Source
 ): Promise<?{ start: any, end: any }> =>
   dispatcher.invoke("getFileGeneratedRange", originalSource);
 
 export const getLocationScopes = dispatcher.task("getLocationScopes");
 
--- a/devtools/client/shared/source-map/index.js
+++ b/devtools/client/shared/source-map/index.js
@@ -135,30 +135,34 @@ const _getGeneratedRanges = dispatcher.t
 
 const _getGeneratedLocation = dispatcher.task("getGeneratedLocation", {
   queue: true
 });
 const _getAllGeneratedLocations = dispatcher.task("getAllGeneratedLocations", {
   queue: true
 });
 
+const _getOriginalLocation = dispatcher.task("getOriginalLocation", {
+  queue: true
+});
+
 const setAssetRootURL = exports.setAssetRootURL = async assetRoot => dispatcher.invoke("setAssetRootURL", assetRoot);
 
 const getOriginalURLs = exports.getOriginalURLs = async generatedSource => dispatcher.invoke("getOriginalURLs", generatedSource);
 
 const hasOriginalURL = exports.hasOriginalURL = async url => dispatcher.invoke("hasOriginalURL", url);
 
 const getOriginalRanges = exports.getOriginalRanges = async (sourceId, url) => dispatcher.invoke("getOriginalRanges", sourceId, url);
 const getGeneratedRanges = exports.getGeneratedRanges = async (location, originalSource) => _getGeneratedRanges(location, originalSource);
 
 const getGeneratedLocation = exports.getGeneratedLocation = async (location, originalSource) => _getGeneratedLocation(location, originalSource);
 
 const getAllGeneratedLocations = exports.getAllGeneratedLocations = async (location, originalSource) => _getAllGeneratedLocations(location, originalSource);
 
-const getOriginalLocation = exports.getOriginalLocation = async (location, options = {}) => dispatcher.invoke("getOriginalLocation", location, options);
+const getOriginalLocation = exports.getOriginalLocation = async (location, options = {}) => _getOriginalLocation(location, options);
 
 const getFileGeneratedRange = exports.getFileGeneratedRange = async originalSource => dispatcher.invoke("getFileGeneratedRange", originalSource);
 
 const getLocationScopes = exports.getLocationScopes = dispatcher.task("getLocationScopes");
 
 const getOriginalSourceText = exports.getOriginalSourceText = async originalSource => dispatcher.invoke("getOriginalSourceText", originalSource);
 
 const applySourceMap = exports.applySourceMap = async (generatedId, url, code, mappings) => dispatcher.invoke("applySourceMap", generatedId, url, code, mappings);