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

Unified Diff: Source/devtools/front_end/bindings/SASSSourceMapping.js

Issue 1328843002: DevTools: introduce WI.SourceMap.Entry structure and cleanup WI.SourceMap API. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: fix test Created 5 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: Source/devtools/front_end/bindings/SASSSourceMapping.js
diff --git a/Source/devtools/front_end/bindings/SASSSourceMapping.js b/Source/devtools/front_end/bindings/SASSSourceMapping.js
index 4cb25c609fb42f73fc9de41228c4b797af2d1393..422b8c3224ce25b1e2a2f6de2d8d2ef6fea054e3 100644
--- a/Source/devtools/front_end/bindings/SASSSourceMapping.js
+++ b/Source/devtools/front_end/bindings/SASSSourceMapping.js
@@ -555,17 +555,16 @@ WebInspector.SASSSourceMapping.prototype = {
*/
rawLocationToUILocation: function(rawLocation)
{
- var entry;
var sourceMap = this._sourceMapByStyleSheetURL[rawLocation.url];
if (!sourceMap)
return null;
- entry = sourceMap.findEntry(rawLocation.lineNumber, rawLocation.columnNumber);
- if (!entry || entry.length === 2)
+ var entry = sourceMap.findEntry(rawLocation.lineNumber, rawLocation.columnNumber);
+ if (!entry || !entry.sourceURL)
return null;
- var uiSourceCode = this._networkMapping.uiSourceCodeForURL(entry[2], rawLocation.target());
+ var uiSourceCode = this._networkMapping.uiSourceCodeForURL(entry.sourceURL, rawLocation.target());
if (!uiSourceCode)
return null;
- return uiSourceCode.uiLocation(entry[3], entry[4]);
+ return uiSourceCode.uiLocation(entry.sourceLineNumber, entry.sourceColumnNumber);
},
/**

Powered by Google App Engine
This is Rietveld 408576698