DevTools: do not detach sourcemap in case of style sheet editing

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

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 242 matching lines @@
         if (!header)
             return original();
 
         var sourceMap = this.sourceMapForHeader(header);
         if (!sourceMap)
             return original();
 
         var originalAndDetach = originalAndDetachIfSuccess.bind(this, header);
 
         if (!sourceMap.editable())
-            return originalAndDetach();
+            return original();
 
         return /** @type {!Promise<boolean>} */(sourceMap.editCompiled([range], [text])
             .then(onEditingDone.bind(this))
             .catch(onError.bind(this, header)));
 
         /**
          * @param {?WebInspector.SourceMap.EditResult} editResult
          * @return {!Promise<boolean>}
          * @this {WebInspector.CSSModel}
          */
@@ skipping 137 matching lines @@
          * @param {?CSSAgent.SelectorList} selectorPayload
          * @return {boolean}
          * @this {WebInspector.CSSModel}
          */
         function callback(error, selectorPayload)
         {
             if (error || !selectorPayload)
                 return false;
             this._domModel.markUndoableState();
             var edit = new WebInspector.CSSModel.Edit(styleSheetId, range, text, selectorPayload);
-            this._fireStyleSheetChangedAndDetach(styleSheetId, edit);
+            this._fireStyleSheetChanged(styleSheetId, edit);
             return true;
         }
 
         WebInspector.userMetrics.actionTaken(WebInspector.UserMetrics.Action.StyleRuleEdited);
         return this._ensureOriginalStyleSheetText(styleSheetId)
             .then(() => this._agent.setRuleSelector(styleSheetId, range, text, callback.bind(this)))
             .catchException(false);
     },
 
     /**
      * @param {!CSSAgent.StyleSheetId} styleSheetId
      * @param {!WebInspector.TextRange} range
      * @param {string} text
      * @return {!Promise<boolean>}
      */
     setKeyframeKey: function(styleSheetId, range, text)
     {
         /**
          * @param {?Protocol.Error} error
          * @param {!CSSAgent.Value} payload
          * @return {boolean}
          * @this {WebInspector.CSSModel}
          */
         function callback(error, payload)
         {
             if (error || !payload)
                 return false;
             this._domModel.markUndoableState();
             var edit = new WebInspector.CSSModel.Edit(styleSheetId, range, text, payload);
-            this._fireStyleSheetChangedAndDetach(styleSheetId, edit);
+            this._fireStyleSheetChanged(styleSheetId, edit);
             return true;
         }
 
         WebInspector.userMetrics.actionTaken(WebInspector.UserMetrics.Action.StyleRuleEdited);
         return this._ensureOriginalStyleSheetText(styleSheetId)
             .then(() => this._agent.setKeyframeKey(styleSheetId, range, text, callback.bind(this)))
             .catchException(false);
     },
 
     /**
@@ skipping 213 matching lines @@
          * @param {!CSSAgent.CSSMedia} mediaPayload
          * @return {boolean}
          * @this {WebInspector.CSSModel}
          */
         function parsePayload(error, mediaPayload)
         {
             if (!mediaPayload)
                 return false;
             this._domModel.markUndoableState();
             var edit = new WebInspector.CSSModel.Edit(styleSheetId, range, newMediaText, mediaPayload);
-            this._fireStyleSheetChangedAndDetach(styleSheetId, edit);
+            this._fireStyleSheetChanged(styleSheetId, edit);
             return true;
         }
 
         WebInspector.userMetrics.actionTaken(WebInspector.UserMetrics.Action.StyleRuleEdited);
         return this._ensureOriginalStyleSheetText(styleSheetId)
             .then(() => this._agent.setMediaText(styleSheetId, range, newMediaText, parsePayload.bind(this)))
             .catchException(false);
     },
 
     /**
@@ skipping 13 matching lines @@
          * @param {?CSSAgent.CSSRule} rulePayload
          * @return {?WebInspector.CSSStyleRule}
          * @this {WebInspector.CSSModel}
          */
         function parsePayload(error, rulePayload)
         {
             if (error || !rulePayload)
                 return null;
             this._domModel.markUndoableState();
             var edit = new WebInspector.CSSModel.Edit(styleSheetId, ruleLocation, ruleText, rulePayload);
-            this._fireStyleSheetChangedAndDetach(styleSheetId, edit);
+            this._fireStyleSheetChanged(styleSheetId, edit);
             return new WebInspector.CSSStyleRule(this, rulePayload);
         }
     },
 
     /**
      * @param {!WebInspector.DOMNode} node
      * @param {function(?WebInspector.CSSStyleSheetHeader)} userCallback
      */
     requestViaInspectorStylesheet: function(node, userCallback)
     {
@@ skipping 49 matching lines @@
      * @param {!CSSAgent.StyleSheetId} styleSheetId
      * @param {!WebInspector.CSSModel.Edit=} edit
      */
     _fireStyleSheetChanged: function(styleSheetId, edit)
     {
         this.dispatchEventToListeners(WebInspector.CSSModel.Events.StyleSheetChanged, { styleSheetId: styleSheetId, edit: edit });
     },
 
     /**
      * @param {!CSSAgent.StyleSheetId} styleSheetId
-     * @param {!WebInspector.CSSModel.Edit=} edit
-     */
-    _fireStyleSheetChangedAndDetach: function(styleSheetId, edit)
-    {
-        this.dispatchEventToListeners(WebInspector.CSSModel.Events.StyleSheetChanged, { styleSheetId: styleSheetId, edit: edit });
-        var header = this.styleSheetHeaderForId(styleSheetId);
-        if (header)
-            this._detachSourceMap(header);
-    },
-
-    /**
-     * @param {!CSSAgent.StyleSheetId} styleSheetId
      * @return {!Promise<string>}
      */
     _ensureOriginalStyleSheetText: function(styleSheetId)
     {
         var header = this.styleSheetHeaderForId(styleSheetId);
         if (!header)
             return Promise.resolve("");
         var promise = this._originalStyleSheetText.get(header);
         if (!promise) {
             promise = this.getStyleSheetText(header.id);
@@ skipping 278 matching lines @@
     {
         this._cssModel.mediaQueryResultChanged();
     },
 
     /**
      * @override
      * @param {!CSSAgent.StyleSheetId} styleSheetId
      */
     styleSheetChanged: function(styleSheetId)
     {
-        this._cssModel._fireStyleSheetChangedAndDetach(styleSheetId);
+        this._cssModel._fireStyleSheetChanged(styleSheetId);
     },
 
     /**
      * @override
      * @param {!CSSAgent.CSSStyleSheetHeader} header
      */
     styleSheetAdded: function(header)
     {
         this._cssModel._styleSheetAdded(header);
     },
@@ skipping 92 matching lines @@
 /**
  * @constructor
  * @param {?WebInspector.CSSStyleDeclaration} inlineStyle
  * @param {?WebInspector.CSSStyleDeclaration} attributesStyle
  */
 WebInspector.CSSModel.InlineStyleResult = function(inlineStyle, attributesStyle)
 {
     this.inlineStyle = inlineStyle;
     this.attributesStyle = attributesStyle;
 }