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

third_party/WebKit/Source/devtools/front_end/sdk/CSSProperty.js

 // Copyright 2016 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
  * @param {!WebInspector.CSSStyleDeclaration} ownerStyle
  * @param {number} index
  * @param {string} name
  * @param {string} value
@@ skipping 10 matching lines @@
     this.index = index;
     this.name = name;
     this.value = value;
     this.important = important;
     this.disabled = disabled;
     this.parsedOk = parsedOk;
     this.implicit = implicit; // A longhand, implicitly set by missing values of shorthand.
     this.text = text;
     this.range = range ? WebInspector.TextRange.fromObject(range) : null;
     this._active = true;
+    this._nameRange = null;
+    this._valueRange = null;
 }
 
 /**
  * @param {!WebInspector.CSSStyleDeclaration} ownerStyle
  * @param {number} index
  * @param {!CSSAgent.CSSProperty} payload
  * @return {!WebInspector.CSSProperty}
  */
 WebInspector.CSSProperty.parsePayload = function(ownerStyle, index, payload)
 {
     // The following default field values are used in the payload:
     // important: false
     // parsedOk: true
     // implicit: false
     // disabled: false
     var result = new WebInspector.CSSProperty(
         ownerStyle, index, payload.name, payload.value, payload.important || false, payload.disabled || false, ("parsedOk" in payload) ? !!payload.parsedOk : true, !!payload.implicit, payload.text, payload.range);
     return result;
 }
 
 WebInspector.CSSProperty.prototype = {
+    _ensureRanges: function()
+    {
+        if (this._nameRange && this._valueRange)
+            return;
+        var range = this.range;
+        var text = this.text ? new WebInspector.Text(this.text) : null;
+        if (!range || !text)
+            return;
+
+        var nameIndex = text.value().indexOf(this.name);
+        var valueIndex = text.value().lastIndexOf(this.value);
+        if (nameIndex === -1 || valueIndex === -1 || nameIndex > valueIndex)
+            return;
+
+        var nameSourceRange = new WebInspector.SourceRange(nameIndex, this.name.length);
+        var valueSourceRange = new WebInspector.SourceRange(valueIndex, this.value.length);
+
+        this._nameRange = rebase(text.toTextRange(nameSourceRange), range.startLine, range.startColumn);
+        this._valueRange = rebase(text.toTextRange(valueSourceRange), range.startLine, range.startColumn);
+
+        /**
+         * @param {!WebInspector.TextRange} oneLineRange
+         * @param {number} lineOffset
+         * @param {number} columnOffset
+         * @return {!WebInspector.TextRange}
+         */
+        function rebase(oneLineRange, lineOffset, columnOffset)
+        {
+            if (oneLineRange.startLine === 0) {
+                oneLineRange.startColumn += columnOffset;
+                oneLineRange.endColumn += columnOffset;
+            }
+            oneLineRange.startLine += lineOffset;
+            oneLineRange.endLine += lineOffset;
+            return oneLineRange;
+        }
+    },
+
+    /**
+     * @return {?WebInspector.TextRange}
+     */
+    nameRange: function()
+    {
+        this._ensureRanges();
+        return this._nameRange;
+    },
+
+    /**
+     * @return {?WebInspector.TextRange}
+     */
+    valueRange: function()
+    {
+        this._ensureRanges();
+        return this._valueRange;
+    },
+
     /**
      * @param {!WebInspector.CSSModel.Edit} edit
      */
     rebase: function(edit)
     {
         if (this.ownerStyle.styleSheetId !== edit.styleSheetId)
             return;
         if (this.range)
             this.range = this.range.rebaseAfterTextEdit(edit.oldRange, edit.newRange);
     },
@@ skipping 171 matching lines @@
     {
         if (!this.ownerStyle)
             return Promise.resolve(false);
         if (disabled === this.disabled)
             return Promise.resolve(true);
         var propertyText = this.text.trim();
         var text = disabled ? "/* " + propertyText + " */" : this.text.substring(2, propertyText.length - 2).trim();
         return this.setText(text, true, true);
     }
 }