Chromium Code Reviews| Index: Source/devtools/front_end/sdk/SASSSourceMapping.js |
| diff --git a/Source/devtools/front_end/sdk/SASSSourceMapping.js b/Source/devtools/front_end/sdk/SASSSourceMapping.js |
| index a48335a610ddc770653c1a76f6fa4cc3140a8ae3..6f0d4b91091720139f22636d614ec6fe04226810 100644 |
| --- a/Source/devtools/front_end/sdk/SASSSourceMapping.js |
| +++ b/Source/devtools/front_end/sdk/SASSSourceMapping.js |
| @@ -419,7 +419,7 @@ WebInspector.SASSSourceMapping.prototype = { |
| var completeSourceMapURL = WebInspector.ParsedURL.completeURL(sourceURL, header.sourceMapURL); |
| if (completeSourceMapURL) |
| delete this._sourceMapByURL[completeSourceMapURL]; |
| - header.updateLocations(); |
| + WebInspector.cssWorkspaceBinding.updateLocations(header); |
| }, |
| /** |
| @@ -445,7 +445,7 @@ WebInspector.SASSSourceMapping.prototype = { |
| this._sourceMapByStyleSheetURL[sourceURL] = sourceMap; |
| for (var i = 0; i < headersWithSameSourceURL.length; ++i) { |
| if (forceRebind) |
| - headersWithSameSourceURL[i].updateLocations(); |
| + WebInspector.cssWorkspaceBinding.updateLocations(headersWithSameSourceURL[i]); |
| else |
| this._bindUISourceCode(headersWithSameSourceURL[i], sourceMap); |
| } |
| @@ -519,7 +519,7 @@ WebInspector.SASSSourceMapping.prototype = { |
| */ |
| _bindUISourceCode: function(header, sourceMap) |
| { |
| - header.pushSourceMapping(this); |
| + WebInspector.cssWorkspaceBinding.pushSourceMapping(header, this); |
| var rawURL = header.sourceURL; |
| var sources = sourceMap.sources(); |
| for (var i = 0; i < sources.length; ++i) { |
| @@ -603,8 +603,9 @@ WebInspector.SASSSourceMapping.prototype = { |
| var ids = this._cssModel.styleSheetIdsForURL(cssURLs[i]); |
| for (var j = 0; j < ids.length; ++j) { |
| var header = this._cssModel.styleSheetHeaderForId(ids[j]); |
| - console.assert(header); |
| - header.updateLocations(); |
| + if (!header) |
| + throw new Error("Header not found for id=" + ids[j]); |
|
vsevik
2014/07/22 14:59:01
This should never happen and we used to assert in
apavlov
2014/07/22 16:19:53
Done.
|
| + WebInspector.cssWorkspaceBinding.updateLocations(header); |
| } |
| } |
| }, |