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

Unified Diff: third_party/WebKit/Source/devtools/front_end/bindings/BlackboxManager.js

Issue 1688283002: [DevTools] Blackboxing in LiveLocations is supported in Linkifier (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@add-source-map-support-v3
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
Index: third_party/WebKit/Source/devtools/front_end/bindings/BlackboxManager.js
diff --git a/third_party/WebKit/Source/devtools/front_end/bindings/BlackboxManager.js b/third_party/WebKit/Source/devtools/front_end/bindings/BlackboxManager.js
index 4fcfa0c67fda193ab15d0aa1c182e0e8b8402767..d77f3c0fa54030cca090fd3b8e0d00f1e366205b 100644
--- a/third_party/WebKit/Source/devtools/front_end/bindings/BlackboxManager.js
+++ b/third_party/WebKit/Source/devtools/front_end/bindings/BlackboxManager.js
@@ -21,9 +21,27 @@ WebInspector.BlackboxManager = function(debuggerWorkspaceBinding, networkMapping
this._scripts = new Map();
/** @type {!Map<string, boolean>} */
this._isBlackboxedURLCache = new Map();
+
+ /** @type {!Set<!WebInspector.DebuggerWorkspaceBinding.Location>} */
+ this._locations = new Set();
}
WebInspector.BlackboxManager.prototype = {
+ /**
+ * @param {!WebInspector.DebuggerWorkspaceBinding.Location} location
+ */
+ addLiveLocation: function(location)
+ {
+ this._locations.add(location);
+ },
+
+ /**
+ * @param {!WebInspector.DebuggerWorkspaceBinding.Location} location
+ */
+ removeLiveLocation: function(location)
+ {
+ this._locations.delete(location);
+ },
/**
* @param {function(!WebInspector.Event)} listener
@@ -299,6 +317,11 @@ WebInspector.BlackboxManager.prototype = {
this._scripts.set(script.scriptId, positions);
else if (!this._scripts.has(script.scriptId))
this._scripts.set(script.scriptId, []);
+
+ for (var location of this._locations) {
+ if (location.script().scriptId === script.scriptId)
+ location.update();
dgozman 2016/02/17 22:46:22 Let's check that DebuggerWorkspaceBinding already
kozy 2016/02/18 01:52:24 Done.
+ }
},
/**

Powered by Google App Engine
This is Rietveld 408576698