DevTools: Strip sourceURL comment from stylesheets before showing them in the editor.

Source/devtools/front_end/CSSStyleModel.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 539 matching lines @@
     },
 
     /**
      * @param {!CSSAgent.StyleSheetId} styleSheetId
      * @param {string} newText
      * @param {boolean} majorChange
      * @param {function(?Protocol.Error)} userCallback
      */
     setStyleSheetText: function(styleSheetId, newText, majorChange, userCallback)
     {
+        var header = this._styleSheetIdToHeader[styleSheetId];
+        console.assert(header);
+        this._pendingCommandsMajorState.push(majorChange);
+        header.setContent(styleSheetId, newText, callback.bind(this));
+
         /**
          * @param {?Protocol.Error} error
          * @this {WebInspector.CSSStyleModel}
          */
         function callback(error)
         {
             this._pendingCommandsMajorState.pop();
             if (!error && majorChange)
                 WebInspector.domAgent.markUndoableState();
             
             if (!error && userCallback)
                 userCallback(error);
         }
-        this._pendingCommandsMajorState.push(majorChange);
-        CSSAgent.setStyleSheetText(styleSheetId, newText, callback.bind(this));
     },
 
     _undoRedoRequested: function()
     {
         this._pendingCommandsMajorState.push(true);
     },
 
     _undoRedoCompleted: function()
     {
         this._pendingCommandsMajorState.pop();
@@ skipping 821 matching lines @@
     this.disabled = payload.disabled;
     this.isInline = payload.isInline;
     this.startLine = payload.startLine;
     this.startColumn = payload.startColumn;
     /** @type {!Set.<!WebInspector.CSSStyleModel.LiveLocation>} */
     this._locations = new Set();
     /** @type {!Array.<!WebInspector.SourceMapping>} */
     this._sourceMappings = [];
 }
 
+WebInspector.CSSStyleSheetHeader.sourceURLRegex = /\n[\040\t]*\/\*[#@][\040\t]sourceURL=[\040\t]*([^\s]*)[\040\t]*\*\/[\040\t]*$/m;

[apavlov, 2013/12/20 10:06:25]

Perhaps, \n -> ^ in the multiline mode? (this may fail to check if the match is
not the first line, though)

+
 WebInspector.CSSStyleSheetHeader.prototype = {
     /**
      * @return {string}
      */
     resourceURL: function()
     {
         return this.origin === "inspector" ? this._viaInspectorResourceURL() : this.sourceURL;
     },
 
     /**
@@ skipping 80 matching lines @@
      * @param {number} columnNumberInStyleSheet
      * @return {number|undefined}
      */
     columnNumberInSource: function(lineNumberInStyleSheet, columnNumberInStyleSheet)
     {
         return (lineNumberInStyleSheet ? 0 : this.startColumn) + columnNumberInStyleSheet;
     },
 
     /**
      * @override
+     * @return {string}
      */
     contentURL: function()
     {
         return this.resourceURL();
     },
 
     /**
      * @override
+     * @return {!WebInspector.ResourceType}
      */
     contentType: function()
     {
         return WebInspector.resourceTypes.Stylesheet;
     },
 
     /**
      * @override
+     * @param {function(?string)} callback
      */
     requestContent: function(callback)
     {
         CSSAgent.getStyleSheetText(this.id, textCallback.bind(this));
 
         /**
          * @this {WebInspector.CSSStyleSheetHeader}
          */
         function textCallback(error, text)
         {
             if (error) {
                 WebInspector.log("Failed to get text for stylesheet " + this.id + ": " + error);
                 text = "";
                 // Fall through.
             }
+            text = text.replace(WebInspector.CSSStyleSheetHeader.sourceURLRegex, "");
             callback(text);
         }
     },
 
     /**
      * @override
      */
     searchInContent: function(query, caseSensitive, isRegex, callback)
     {
         function performSearch(content)
         {
             callback(WebInspector.ContentProvider.performSearchInContent(content, query, caseSensitive, isRegex));
         }
 
         // searchInContent should call back later.
         this.requestContent(performSearch);
-    }
+    },
+
+    /**
+     * @param {!CSSAgent.StyleSheetId} styleSheetId
+     * @param {string} newText
+     * @param {function(?Protocol.Error)} callback
+     */
+    setContent: function(styleSheetId, newText, callback)
+    {
+        var regex = WebInspector.CSSStyleSheetHeader.sourceURLRegex;

[apavlov, 2013/12/20 10:06:25]

This can be inlined, since regex is used only once below

+        if (this.hasSourceURL && !regex.exec(newText))

[apavlov, 2013/12/20 10:06:25]

exec->test

+            newText += "\n/*# sourceURL=" + this.sourceURL + " */";
+        CSSAgent.setStyleSheetText(styleSheetId, newText, callback);
+    },
 }
 
 /**
  * @constructor
  * @param {!CSSAgent.StyleSheetId} styleSheetId
  * @param {!Array.<!CSSAgent.CSSRule>} payload
  */
 WebInspector.CSSStyleSheet = function(styleSheetId, payload)
 {
     this.id = styleSheetId;
@@ skipping 225 matching lines @@
             for (var i = 0; i < callbacks.length; ++i)
                 callbacks[i](computedStyle);
         }
     }
 }
 
 /**
  * @type {!WebInspector.CSSStyleModel}
  */
 WebInspector.cssModel;