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

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

Issue 299443016: DevTools: Decouple debugger model from UI entities (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Merge DebuggerScriptMapping into DebuggerWorkspaceBinding Created 6 years, 5 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/ResourceScriptMapping.js
diff --git a/Source/devtools/front_end/sdk/ResourceScriptMapping.js b/Source/devtools/front_end/sdk/ResourceScriptMapping.js
index a9565d878e1509c8a71e4a41ca2f518aee1fb355..675144f7a74c6a51b7fd1926dd61920ce4b4bf19 100644
--- a/Source/devtools/front_end/sdk/ResourceScriptMapping.js
+++ b/Source/devtools/front_end/sdk/ResourceScriptMapping.js
@@ -33,13 +33,15 @@
* @implements {WebInspector.ScriptSourceMapping}
* @param {!WebInspector.DebuggerModel} debuggerModel
* @param {!WebInspector.Workspace} workspace
+ * @param {!WebInspector.DebuggerWorkspaceBinding} debuggerWorkspaceBinding
*/
-WebInspector.ResourceScriptMapping = function(debuggerModel, workspace)
+WebInspector.ResourceScriptMapping = function(debuggerModel, workspace, debuggerWorkspaceBinding)
{
this._target = debuggerModel.target();
this._debuggerModel = debuggerModel;
this._workspace = workspace;
this._workspace.addEventListener(WebInspector.Workspace.Events.UISourceCodeAdded, this._uiSourceCodeAddedToWorkspace, this);
+ this._debuggerWorkspaceBinding = debuggerWorkspaceBinding;
this._boundURLs = new StringSet();
debuggerModel.addEventListener(WebInspector.DebuggerModel.Events.GlobalObjectCleared, this._debuggerReset, this);
@@ -83,7 +85,7 @@ WebInspector.ResourceScriptMapping.prototype = {
{
if (script.isAnonymousScript())
return;
- script.pushSourceMapping(this);
+ this._debuggerWorkspaceBinding.pushSourceMapping(script, this);
var uiSourceCode = this._workspaceUISourceCodeForScript(script);
if (!uiSourceCode)
@@ -134,7 +136,7 @@ WebInspector.ResourceScriptMapping.prototype = {
if (!scripts.length)
return;
for (var i = 0; i < scripts.length; ++i)
- scripts[i].updateLocations();
+ this._debuggerWorkspaceBinding.updateLocations(scripts[i]);
},
/**
@@ -146,7 +148,7 @@ WebInspector.ResourceScriptMapping.prototype = {
if (!scripts.length)
return;
for (var i = 0; i < scripts.length; ++i)
- scripts[i].updateLocations();
+ this._debuggerWorkspaceBinding.updateLocations(scripts[i]);
},
/**
@@ -181,7 +183,7 @@ WebInspector.ResourceScriptMapping.prototype = {
var scriptFile = new WebInspector.ResourceScriptFile(this, uiSourceCode, scripts);
uiSourceCode.setScriptFileForTarget(this._target, scriptFile);
for (var i = 0; i < scripts.length; ++i)
- scripts[i].updateLocations();
+ this._debuggerWorkspaceBinding.updateLocations(scripts[i]);
uiSourceCode.setSourceMappingForTarget(this._target, this);
this._boundURLs.add(uiSourceCode.url);
},

Powered by Google App Engine
This is Rietveld 408576698