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

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

Issue 2376583002: DevTools: remove WI.NetworkMapping.networkURL (Closed)
Patch Set: do not blackbox fs uiSourceCodes Created 4 years, 3 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/CompilerScriptMapping.js
diff --git a/third_party/WebKit/Source/devtools/front_end/bindings/CompilerScriptMapping.js b/third_party/WebKit/Source/devtools/front_end/bindings/CompilerScriptMapping.js
index ca159e812afcdc40a2598a1637181ff33b461836..e08e781569aed211ae02f08f9c0214283a4b3cff 100644
--- a/third_party/WebKit/Source/devtools/front_end/bindings/CompilerScriptMapping.js
+++ b/third_party/WebKit/Source/devtools/front_end/bindings/CompilerScriptMapping.js
@@ -129,15 +129,12 @@ WebInspector.CompilerScriptMapping.prototype = {
{
if (uiSourceCode.project().type() === WebInspector.projectTypes.Service)
return null;
- var networkURL = this._networkMapping.networkURL(uiSourceCode);
- if (!networkURL)
- return null;
- var sourceMap = this._sourceMapForURL.get(networkURL);
+ var sourceMap = this._sourceMapForURL.get(uiSourceCode.url());
if (!sourceMap)
return null;
var script = /** @type {!WebInspector.Script} */ (this._scriptForSourceMap.get(sourceMap));
console.assert(script);
- var entry = sourceMap.firstSourceLineMapping(networkURL, lineNumber);
+ var entry = sourceMap.firstSourceLineMapping(uiSourceCode.url(), lineNumber);
if (!entry)
return null;
return this._debuggerModel.createRawLocation(script, entry.lineNumber, entry.columnNumber);
@@ -280,13 +277,10 @@ WebInspector.CompilerScriptMapping.prototype = {
*/
uiLineHasMapping: function(uiSourceCode, lineNumber)
{
- var networkURL = this._networkMapping.networkURL(uiSourceCode);
- if (!networkURL)
- return true;
- var sourceMap = this._sourceMapForURL.get(networkURL);
+ var sourceMap = this._sourceMapForURL.get(uiSourceCode.url());
if (!sourceMap)
return true;
- return !!sourceMap.firstSourceLineMapping(networkURL, lineNumber);
+ return !!sourceMap.firstSourceLineMapping(uiSourceCode.url(), lineNumber);
},
/**
@@ -311,8 +305,7 @@ WebInspector.CompilerScriptMapping.prototype = {
_uiSourceCodeAddedToWorkspace: function(event)
{
var uiSourceCode = /** @type {!WebInspector.UISourceCode} */ (event.data);
- var networkURL = this._networkMapping.networkURL(uiSourceCode);
- if (!networkURL || !this._sourceMapForURL.get(networkURL))
+ if (!this._sourceMapForURL.get(uiSourceCode.url()))
return;
this._bindUISourceCode(uiSourceCode);
},

Powered by Google App Engine
This is Rietveld 408576698