DevTools: [LiveSASS] property enabling/disabling should not add spaces

third_party/WebKit/Source/devtools/front_end/sass/SASSSupport.js

 // Copyright 2015 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.
 
 WebInspector.SASSSupport = {}
 
 /**
  * @param {string} url
  * @param {string} content
  * @return {!Promise<!WebInspector.SASSSupport.AST>}
@@ skipping 232 matching lines @@
         if (disabled) {
             var oldRange1 = WebInspector.TextRange.createFromLocation(this.range.startLine, this.range.startColumn);
             var edit1 = new WebInspector.SourceEdit(this.document.url, oldRange1, "/* ");
             var oldRange2 = WebInspector.TextRange.createFromLocation(this.range.endLine, this.range.endColumn);
             var edit2 = new WebInspector.SourceEdit(this.document.url, oldRange2, " */");
             this.document.edits.push(edit1, edit2);
             return;
         }
         var oldRange1 = new WebInspector.TextRange(this.range.startLine, this.range.startColumn, this.range.startLine, this.name.range.startColumn);
         var edit1 = new WebInspector.SourceEdit(this.document.url, oldRange1, "");
-        var oldRange2 = new WebInspector.TextRange(this.range.endLine, this.range.endColumn - 2, this.range.endLine, this.range.endColumn);
+
+        var propertyText = this.document.text.extract(this.range);
+        var endsWithSemicolon = propertyText.slice(2, -2).trim().endsWith(";");

[dgozman, 2016/05/30 18:27:11]

No need to slice from the start?

[lushnikov, 2016/07/11 18:55:09]

Done.

+        var oldRange2 = new WebInspector.TextRange(this.range.endLine, this.value.range.endColumn + (endsWithSemicolon ? 1 : 0), this.range.endLine, this.range.endColumn);
         var edit2 = new WebInspector.SourceEdit(this.document.url, oldRange2, "");
         this.document.edits.push(edit1, edit2);
     },
 
     remove: function()
     {
         console.assert(this.parent);
         var rule = this.parent;
         var index = rule.properties.indexOf(this);
         rule.properties.splice(index, 1);
@@ skipping 417 matching lines @@
         mapping.set(oldProperty.name, newProperty.name);
         mapping.set(oldProperty.value, newProperty.value);
         if (oldProperty.name.text.trim() !== newProperty.name.text.trim())
             addChange(T.NameChanged, oldRule, newRule, oldPropertyIndex, newPropertyIndex);
         if (oldProperty.value.text.trim() !== newProperty.value.text.trim())
             addChange(T.ValueChanged, oldRule, newRule, oldPropertyIndex, newPropertyIndex);
         if (oldProperty.disabled !== newProperty.disabled)
             addChange(T.PropertyToggled, oldRule, newRule, oldPropertyIndex, newPropertyIndex);
     }
 }