[DevTools] Do not report edited resources via Page.getResourceContent.

Source/devtools/front_end/bindings/ResourceScriptMapping.js

 /*
  * Copyright (C) 2012 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 221 matching lines @@
         return this._debuggerModel.scriptsForSourceURL(this._networkMapping.networkURL(uiSourceCode));
     },
 
     /**
      * @param {!WebInspector.UISourceCode} uiSourceCode
      * @param {!Array.<!WebInspector.Script>} scripts
      */
     _bindUISourceCodeToScripts: function(uiSourceCode, scripts)
     {
         console.assert(scripts.length);
+        var boundScriptFile = this.scriptFile(uiSourceCode);
+        if (boundScriptFile && boundScriptFile.hasScripts(scripts))

[yurys, 2015/07/16 16:30:35]

Was it a bug or what? If so it may make sense to extract them into a separate
CL.

[dgozman, 2015/07/17 14:39:16]

This is a race, but it always happened in one order because of event listeners
order. So, before this patch it was even impossible to write a test.

+            return;
+
         var scriptFile = new WebInspector.ResourceScriptFile(this, uiSourceCode, scripts);
         this._setScriptFile(uiSourceCode, scriptFile);
         for (var i = 0; i < scripts.length; ++i)
             this._debuggerWorkspaceBinding.updateLocations(scripts[i]);
         this._debuggerWorkspaceBinding.setSourceMapping(this._target, uiSourceCode, this);
         this._boundURLs.add(this._networkMapping.networkURL(uiSourceCode));
     },
 
     /**
      * @param {!WebInspector.UISourceCode} uiSourceCode
@@ skipping 51 matching lines @@
     this._update();
 }
 
 WebInspector.ResourceScriptFile.Events = {
     DidMergeToVM: "DidMergeToVM",
     DidDivergeFromVM: "DidDivergeFromVM",
 }
 
 WebInspector.ResourceScriptFile.prototype = {
     /**
+     * @param {!Array.<!WebInspector.Script>} scripts
+     * @return {boolean}
+     */
+    hasScripts: function(scripts)
+    {
+        var script;
+        if (this._uiSourceCode.contentType() === WebInspector.resourceTypes.Script)
+            script = scripts[0];
+        return this._script === script;
+    },
+
+    /**
      * @param {function(?string,!DebuggerAgent.SetScriptSourceError=,!WebInspector.Script=)=} callback
      */
     commitLiveEdit: function(callback)
     {
         /**
          * @param {?string} error
          * @param {!DebuggerAgent.SetScriptSourceError=} errorData
          * @this {WebInspector.ResourceScriptFile}
          */
         function innerCallback(error, errorData)
@@ skipping 125 matching lines @@
      */
     addSourceMapURL: function(sourceMapURL)
     {
         if (!this._script)
             return;
         this._script.addSourceMapURL(sourceMapURL);
     },
 
     __proto__: WebInspector.Object.prototype
 }