Chromium Code Reviews

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

Issue 475803002: Make profiling lock global rather than per Target (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Index: Source/devtools/front_end/sdk/DOMModel.js
diff --git a/Source/devtools/front_end/sdk/DOMModel.js b/Source/devtools/front_end/sdk/DOMModel.js
index 0d3f0d3a33ffa9e3346fafacb5aa45c922ae3172..25f2a2ffe2e1e3a954a7580b2f0ab3ed2557afe9 100644
--- a/Source/devtools/front_end/sdk/DOMModel.js
+++ b/Source/devtools/front_end/sdk/DOMModel.js
@@ -949,7 +949,7 @@ WebInspector.DOMModel = function(target) {
this._highlighter = this._defaultHighlighter;
if (WebInspector.experimentsSettings.disableAgentsWhenProfile.isEnabled())
- target.profilingLock.addEventListener(WebInspector.Lock.Events.StateChanged, this._profilingStateChanged, this);
+ WebInspector.profilingLock.addEventListener(WebInspector.Lock.Events.StateChanged, this._profilingStateChanged, this);
this._agent.enable();
}
@@ -970,7 +970,7 @@ WebInspector.DOMModel.Events = {
WebInspector.DOMModel.prototype = {
_profilingStateChanged: function()
{
- if (this.target().profilingLock.isAcquired())
+ if (WebInspector.profilingLock.isAcquired())
this._agent.disable();
else
this._agent.enable();

Powered by Google App Engine