DevTools: make StyleSourceMapping in charge of managing UISourceCodes

third_party/WebKit/Source/devtools/front_end/bindings/CSSWorkspaceBinding.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @implements {SDK.SDKModelObserver<!SDK.CSSModel>}
  */
 Bindings.CSSWorkspaceBinding = class {
   /**
    * @param {!SDK.TargetManager} targetManager
@@ skipping 122 matching lines @@
    */
   uiLocationToRawLocations(uiLocation) {},
 };
 
 Bindings.CSSWorkspaceBinding.ModelInfo = class {
   /**
    * @param {!SDK.CSSModel} cssModel
    * @param {!Workspace.Workspace} workspace
    */
   constructor(cssModel, workspace) {
+    this._workspace = workspace;
     this._eventListeners = [
       cssModel.addEventListener(SDK.CSSModel.Events.StyleSheetAdded, this._styleSheetAdded, this),
       cssModel.addEventListener(SDK.CSSModel.Events.StyleSheetRemoved, this._styleSheetRemoved, this)
     ];
 
     this._stylesSourceMapping = new Bindings.StylesSourceMapping(cssModel, workspace);
     var sourceMapManager = cssModel.sourceMapManager();
     this._sassSourceMapping = new Bindings.SASSSourceMapping(cssModel.target(), sourceMapManager, workspace);
 
     /** @type {!Multimap<!SDK.CSSStyleSheetHeader, !Bindings.CSSWorkspaceBinding.LiveLocation>} */
@@ skipping 69 matching lines @@
   }
 
   /**
    * @param {!SDK.CSSLocation} rawLocation
    * @return {?Workspace.UILocation}
    */
   _rawLocationToUILocation(rawLocation) {
     var uiLocation = null;
     uiLocation = uiLocation || this._sassSourceMapping.rawLocationToUILocation(rawLocation);
     uiLocation = uiLocation || this._stylesSourceMapping.rawLocationToUILocation(rawLocation);
-    return uiLocation;
+    if (uiLocation)
+      return uiLocation;
+    var header = rawLocation.header();
+    if (!header)
+      return null;
+    var uiSourceCode = Bindings.NetworkProject.uiSourceCodeForResourceURL(

[dgozman, 2017/05/17 16:46:20]

Should we have a ResourceMapping which will do this instead? It can register
itself as mapping both in JS and CSS worlds, and only resolve locations but not
perform any content syncing.

[lushnikov, 2017/05/20 01:24:18]

Done - introduced here: https://codereview.chromium.org/2893073002/

+        header.cssModel().target(), this._workspace, rawLocation.url, header.frameId);
+    if (!uiSourceCode)
+      return null;
+    return uiSourceCode.uiLocation(rawLocation.lineNumber, rawLocation.columnNumber);
   }
 
   /**
    * @param {!Workspace.UILocation} uiLocation
    * @return {!Array<!SDK.CSSLocation>}
    */
   _uiLocationToRawLocations(uiLocation) {
     var rawLocations = this._sassSourceMapping.uiLocationToRawLocations(uiLocation);
     if (rawLocations.length)
       return rawLocations;
@@ skipping 52 matching lines @@
    */
   isBlackboxed() {
     return false;
   }
 };
 
 /**
  * @type {!Bindings.CSSWorkspaceBinding}
  */
 Bindings.cssWorkspaceBinding;