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

Source/devtools/front_end/bindings/NetworkProject.js

Index: Source/devtools/front_end/bindings/NetworkProject.js
diff --git a/Source/devtools/front_end/bindings/NetworkProject.js b/Source/devtools/front_end/bindings/NetworkProject.js
index 33364e77c63400490bb9d0ee2f4217dac6eea841..8c58bc02ddbd1e75e08d334ae1c202b3036634a2 100644
--- a/Source/devtools/front_end/bindings/NetworkProject.js
+++ b/Source/devtools/front_end/bindings/NetworkProject.js
@@ -244,6 +244,15 @@ WebInspector.NetworkProject.prototype = {
     },
 
     /**
+     * @param {!WebInspector.Resource} resource
+     * @return {!WebInspector.ContentProvider}
+     */
+    _contentProviderForResource: function(resource)
+    {
+        return resource.resourceType() === WebInspector.resourceTypes.Document ? new WebInspector.NetworkProject.DocumentFallbackResource(resource) : resource;

[pfeldman, 2015/08/11 23:22:43]

Lets use this opportunity to nuke the concatenated resource. All you need here
is a plain HTML resource.

[dgozman, 2015/08/12 23:22:11]

Done.

+    },
+
+    /**
      * @param {string} url
      * @param {!WebInspector.ContentProvider} contentProvider
      * @param {boolean=} isContentScript
@@ -287,7 +296,7 @@ WebInspector.NetworkProject.prototype = {
 
             var resources = frame.resources();
             for (var i = 0; i < resources.length; ++i)
-                this._addFile(resources[i].url, new WebInspector.NetworkProject.FallbackResource(resources[i]));
+                this._addFile(resources[i].url, this._contentProviderForResource(resources[i]));
         }
 
         var mainFrame = this.target().resourceTreeModel.mainFrame;
@@ -342,7 +351,9 @@ WebInspector.NetworkProject.prototype = {
     _resourceAdded: function(event)
     {
         var resource = /** @type {!WebInspector.Resource} */ (event.data);
-        this._addFile(resource.url, new WebInspector.NetworkProject.FallbackResource(resource));
+        if (resource.resourceType() === WebInspector.resourceTypes.Script || resource.resourceType() === WebInspector.resourceTypes.Stylesheet)

[pfeldman, 2015/08/11 23:22:43]

Why would we do anything upon resourceAdded? I thought we agreed upon nuking it
altogether.

[dgozman, 2015/08/12 23:22:11]

Done.

+            return;
+        this._addFile(resource.url, this._contentProviderForResource(resource));
     },
 
     /**
@@ -419,12 +430,13 @@ WebInspector.NetworkProject.prototype = {
  * @implements {WebInspector.ContentProvider}
  * @param {!WebInspector.Resource} resource
  */
-WebInspector.NetworkProject.FallbackResource = function(resource)
+WebInspector.NetworkProject.DocumentFallbackResource = function(resource)
 {
+    console.assert(resource.resourceType() === WebInspector.resourceTypes.Document);
     this._resource = resource;
 }
 
-WebInspector.NetworkProject.FallbackResource.prototype = {
+WebInspector.NetworkProject.DocumentFallbackResource.prototype = {
 
     /**
      * @override
@@ -451,46 +463,27 @@ WebInspector.NetworkProject.FallbackResource.prototype = {
     requestContent: function(callback)
     {
         /**
-         * @this {WebInspector.NetworkProject.FallbackResource}
+         * @param {?string} content
+         * @this {WebInspector.NetworkProject.DocumentFallbackResource}
          */
-        function loadFallbackContent()
+        function loadFallbackContent(content)
         {
-            var debuggerModel = WebInspector.DebuggerModel.fromTarget(this._resource.target());
-            if (!debuggerModel) {
-                callback(null);
+            if (content) {
+                callback(content);
                 return;
             }
-            var scripts = debuggerModel.scriptsForSourceURL(this._resource.url);
+
+            var debuggerModel = WebInspector.DebuggerModel.fromTarget(this._resource.target());
+            var scripts = debuggerModel ? debuggerModel.scriptsForSourceURL(this._resource.url) : [];
             if (!scripts.length) {
                 callback(null);
                 return;
             }
 
-            var contentProvider;
-            var type = this._resource.resourceType();
-            if (type === WebInspector.resourceTypes.Document)
-                contentProvider = new WebInspector.ConcatenatedScriptsContentProvider(scripts);
-            else if (type === WebInspector.resourceTypes.Script)
-                contentProvider = scripts[0];
-
-            console.assert(contentProvider, "Resource content request failed. " + this._resource.url);
-
-            contentProvider.requestContent(callback);
+            new WebInspector.ConcatenatedScriptsContentProvider(scripts).requestContent(callback);
         }
 
-        /**
-         * @param {?string} content
-         * @this {WebInspector.NetworkProject.FallbackResource}
-         */
-        function requestContentLoaded(content)
-        {
-            if (content)
-                callback(content)
-            else
-                loadFallbackContent.call(this);
-        }
-
-        this._resource.requestContent(requestContentLoaded.bind(this));
+        this._resource.requestContent(loadFallbackContent.bind(this));
     },
 
     /**
@@ -516,11 +509,6 @@ WebInspector.NetworkProject.FallbackResource.prototype = {
             callback(result);
         }
 
-        if (this.contentType() === WebInspector.resourceTypes.Document) {
-            this.requestContent(documentContentLoaded);
-            return;
-        }
-
-        this._resource.searchInContent(query, caseSensitive, isRegex, callback);
+        this.requestContent(documentContentLoaded);
     }
 }