Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(831)

Unified Diff: third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js

Issue 1663723002: [DevTools] Add sourceMap support for blackboxing (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@call-set-blackboxed-ranges-on-script-parsed
Patch Set: Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/WebKit/Source/devtools/front_end/sources/SourcesPanel.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js
diff --git a/third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js b/third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js
index 0230f62b76e0109bd9d00b3cba7d99e0f276ed24..34e507b9e77697020c8695bc087e10cdf8b1e48a 100644
--- a/third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js
+++ b/third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js
@@ -347,8 +347,6 @@ WebInspector.TimelineFlameChartDataProvider.prototype = {
/** @type {!Array.<!WebInspector.TimelineFlameChartMarker>} */
this._markers = [];
this._asyncColorByCategory = {};
- /** @type {!Map<string, boolean>} */
- this._blackboxingURLCache = new Map();
},
/**
@@ -473,11 +471,7 @@ WebInspector.TimelineFlameChartDataProvider.prototype = {
*/
_isBlackboxedURL: function(url)
{
- if (this._blackboxingURLCache.has(url))
- return /** @type {boolean} */ (this._blackboxingURLCache.get(url));
- var result = WebInspector.BlackboxSupport.isBlackboxedURL(url);
- this._blackboxingURLCache.set(url, result);
- return result;
+ return WebInspector.blackboxManager.isBlackboxedURL(url);
},
/**
@@ -1255,7 +1249,7 @@ WebInspector.TimelineFlameChartView = function(delegate, timelineModel, frameMod
this._model.addEventListener(WebInspector.TimelineModel.Events.RecordingStarted, this._onRecordingStarted, this);
this._mainView.addEventListener(WebInspector.FlameChart.Events.EntrySelected, this._onMainEntrySelected, this);
this._networkView.addEventListener(WebInspector.FlameChart.Events.EntrySelected, this._onNetworkEntrySelected, this);
- WebInspector.BlackboxSupport.addChangeListener(this.refreshRecords, this);
+ WebInspector.blackboxManager.addChangeListener(this.refreshRecords, this);
}
WebInspector.TimelineFlameChartView.prototype = {
@@ -1267,7 +1261,7 @@ WebInspector.TimelineFlameChartView.prototype = {
this._model.removeEventListener(WebInspector.TimelineModel.Events.RecordingStarted, this._onRecordingStarted, this);
this._mainView.removeEventListener(WebInspector.FlameChart.Events.EntrySelected, this._onMainEntrySelected, this);
this._networkView.removeEventListener(WebInspector.FlameChart.Events.EntrySelected, this._onNetworkEntrySelected, this);
- WebInspector.BlackboxSupport.removeChangeListener(this.refreshRecords, this);
+ WebInspector.blackboxManager.removeChangeListener(this.refreshRecords, this);
},
/**
« no previous file with comments | « third_party/WebKit/Source/devtools/front_end/sources/SourcesPanel.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698