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

Unified Diff: Source/devtools/front_end/sdk/Target.js

Issue 310463003: DevTools: introduce TargetBreakpoints as a presentation of breakpoint and its state within target (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebase on master Created 6 years, 6 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: Source/devtools/front_end/sdk/Target.js
diff --git a/Source/devtools/front_end/sdk/Target.js b/Source/devtools/front_end/sdk/Target.js
index 5fd831a0c6c3b7b0c736501cfb8a498d1972b8f3..5435944f666cb337ac09b1c8a2f5963d19557300 100644
--- a/Source/devtools/front_end/sdk/Target.js
+++ b/Source/devtools/front_end/sdk/Target.js
@@ -231,11 +231,19 @@ WebInspector.TargetManager.prototype = {
*/
observeTargets: function(targetObserver)
{
- WebInspector.targetManager.targets().forEach(targetObserver.targetAdded.bind(targetObserver));
+ this.targets().forEach(targetObserver.targetAdded.bind(targetObserver));
this._observers.push(targetObserver);
},
/**
+ * @param {!WebInspector.TargetManager.Observer} targetObserver
+ */
+ removeTargetObserver: function(targetObserver)
vsevik 2014/06/04 16:28:53 unobserveTargets or stopObservingTargets for consi
sergeyv 2014/06/05 12:59:51 Done.
+ {
+ this._observers.remove(targetObserver);
+ },
+
+ /**
* @param {!InspectorBackendClass.Connection} connection
* @param {function(!WebInspector.Target)=} callback
*/

Powered by Google App Engine
This is Rietveld 408576698