DevTools: [SASS] fix CSSWorkspaceBinding.propertyUILocation to account for comments

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.
 
 /**
  * @constructor
  * @implements {WebInspector.TargetManager.Observer}
  * @param {!WebInspector.TargetManager} targetManager
  * @param {!WebInspector.Workspace} workspace
  * @param {!WebInspector.NetworkMapping} networkMapping
@@ skipping 118 matching lines @@
     },
 
     /**
      * @param {!WebInspector.CSSProperty} cssProperty
      * @param {boolean} forName
      * @return {?WebInspector.UILocation}
      */
     propertyUILocation: function(cssProperty, forName)
     {
         var style = cssProperty.ownerStyle;
-        if (!style || !style.parentRule || !style.styleSheetId)
+        if (!style || style.type !== WebInspector.CSSStyleDeclaration.Type.Regular || !style.styleSheetId)
             return null;
-
-        var range = cssProperty.range;
-        if (!range)
-            return null;
-
         var header = style.cssModel().styleSheetHeaderForId(style.styleSheetId);
         if (!header)
             return null;
 
-        var line = forName ? range.startLine : range.endLine;
-        // End of range is exclusive, so subtract 1 from the end offset.
-        var column = forName ? range.startColumn : range.endColumn - (cssProperty.text && cssProperty.text.endsWith(";") ? 2 : 1);
-        var rawLocation = new WebInspector.CSSLocation(header, header.lineNumberInSource(line), header.columnNumberInSource(line, column));
+        var range = forName ? cssProperty.nameRange() : cssProperty.valueRange();
+        if (!range)
+            return null;
+
+        var lineNumber = range.startLine;
+        var columnNumber = range.startColumn;
+        var rawLocation = new WebInspector.CSSLocation(header, header.lineNumberInSource(lineNumber), header.columnNumberInSource(lineNumber, columnNumber));
         return this.rawLocationToUILocation(rawLocation);
     },
 
     /**
      * @param {?WebInspector.CSSLocation} rawLocation
      * @return {?WebInspector.UILocation}
      */
     rawLocationToUILocation: function(rawLocation)
     {
         if (!rawLocation)
@@ skipping 286 matching lines @@
      * @param {number} lineNumber
      * @return {boolean}
      */
     uiLineHasMapping: function(uiSourceCode, lineNumber) { }
 }
 
 /**
  * @type {!WebInspector.CSSWorkspaceBinding}
  */
 WebInspector.cssWorkspaceBinding;