DevTools: remove illusionary caching from String.prototype.lineEndings

third_party/WebKit/Source/devtools/front_end/sass/SASSProcessor.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.ASTService} astService
  * @param {!WebInspector.ASTSourceMap} map
  * @param {!Array<!WebInspector.SASSProcessor.EditOperation>} editOperations
  */
@@ skipping 17 matching lines @@
             changedCSSRules.addAll(rules);
         }
 
         // Reparse new texts, make sure changes result in anticipated AST trees.
         var promises = [];
         for (var ast of this._map.models().values()) {
             if (!ast.document.hasChanged())
                 continue;
             var promise;
             if (ast.document.url === this._map.cssURL())
-                promise = this._astService.parseCSS(ast.document.url, ast.document.newText());
+                promise = this._astService.parseCSS(ast.document.url, ast.document.newText().value());
             else
-                promise = this._astService.parseSCSS(ast.document.url, ast.document.newText());
+                promise = this._astService.parseSCSS(ast.document.url, ast.document.newText().value());
             promises.push(promise);
         }
 
         return Promise.all(promises)
             .then(this._onFinished.bind(this, changedCSSRules));
     },
 
     /**
      * @param {!Set<!WebInspector.SASSSupport.Rule>} changedCSSRules
      * @param {!Array<!WebInspector.SASSSupport.AST>} changedModels
      * @return {?WebInspector.SASSProcessor.Result}
      */
     _onFinished: function(changedCSSRules, changedModels)
     {
         var nodeMapping = new Map();
         var map = this._map.rebase(changedModels, nodeMapping);
         if (!map)
             return null;
 
         var cssEdits = [];
         for (var rule of changedCSSRules) {
             var oldRange = rule.styleRange;
             var newRule = nodeMapping.get(rule);
-            var newText = newRule.styleRange.extract(newRule.document.text);
+            var newText = newRule.document.text.extract(newRule.styleRange);
             cssEdits.push(new WebInspector.SourceEdit(newRule.document.url, oldRange, newText));
         }
 
         /** @type {!Map<string, string>} */
         var newSASSSources = new Map();
         for (var model of changedModels) {
             if (model.document.url === map.cssURL())
                 continue;
-            newSASSSources.set(model.document.url, model.document.text);
+            newSASSSources.set(model.document.url, model.document.text.value());
         }
         return new WebInspector.SASSProcessor.Result(map, cssEdits, newSASSSources);
     }
 }
 
 /**
  * @constructor
  * @param {!WebInspector.ASTSourceMap} map
  * @param {!Array<!WebInspector.SourceEdit>} cssEdits
  * @param {!Map<string, string>} newSASSSources
@@ skipping 10 matching lines @@
  * @param {!WebInspector.ASTSourceMap} map
  * @param {!Array<!WebInspector.TextRange>} ranges
  * @param {!Array<string>} newTexts
  * @return {!Promise<?WebInspector.SASSProcessor.Result>}
  */
 WebInspector.SASSProcessor.processCSSEdits = function(astService, map, ranges, newTexts)
 {
     console.assert(ranges.length === newTexts.length);
     var cssURL = map.cssURL();
     var cssText = map.cssAST().document.text;
-    for (var i = 0; i < ranges.length; ++i) {
-        var range = ranges[i];
-        var edit = new WebInspector.SourceEdit(cssURL, range, newTexts[i]);
-        cssText = edit.applyToText(cssText);
-    }
-    return astService.parseCSS(cssURL, cssText)
+    for (var i = 0; i < ranges.length; ++i)
+        cssText = new WebInspector.Text(cssText.replaceRange(ranges[i], newTexts[i]));
+    return astService.parseCSS(cssURL, cssText.value())
         .then(onCSSParsed);
 
     /**
      * @param {!WebInspector.SASSSupport.AST} newCSSAST
      * @return {!Promise<?WebInspector.SASSProcessor.Result>}
      */
     function onCSSParsed(newCSSAST)
     {
         //TODO(lushnikov): only diff changed styles.
         var cssDiff = WebInspector.SASSSupport.diffModels(map.cssAST(), newCSSAST);
@@ skipping 419 matching lines @@
      * @return {!WebInspector.SASSProcessor.InsertPropertiesOperation}
      */
     rebase: function(newMap, nodeMapping)
     {
         var sassAnchor = /** @type {?WebInspector.SASSSupport.Property} */(nodeMapping.get(this._sassAnchor)) || this._sassAnchor;
         return new WebInspector.SASSProcessor.InsertPropertiesOperation(newMap, sassAnchor, this._insertBefore, this._nameTexts, this._valueTexts, this._disabledStates);
     },
 
     __proto__: WebInspector.SASSProcessor.EditOperation.prototype
 }