DevTools: support targets that don't have Resource agent

Source/devtools/front_end/sdk/ResourceTreeModel.js

Index: Source/devtools/front_end/sdk/ResourceTreeModel.js
diff --git a/Source/devtools/front_end/sdk/ResourceTreeModel.js b/Source/devtools/front_end/sdk/ResourceTreeModel.js
index 1d10f696138da4c7133c3fa53ef305cf3974b2ca..2d8ac60e027992b880c753bbcfa195c0b3908980 100644
--- a/Source/devtools/front_end/sdk/ResourceTreeModel.js
+++ b/Source/devtools/front_end/sdk/ResourceTreeModel.js
@@ -107,20 +107,17 @@ WebInspector.ResourceTreeModel.prototype = {
     {
         /** @type {!Object.<string, !WebInspector.ResourceTreeFrame>} */
         this._frames = {};
-
-        if (this.target().isDedicatedWorker()) {
-            this._cachedResourcesProcessed = true;
-            return;
-        }
-
-        delete this._cachedResourcesProcessed;
+        this._cachedResourcesProcessed = false;
         this._agent.getResourceTree(this._processCachedResources.bind(this));
     },
 
     _processCachedResources: function(error, mainFramePayload)
     {
         if (error) {
-            console.error(JSON.stringify(error));
+            // FIXME: support targets that don't have resourceTreeModel.

[pfeldman, 2015/07/02 17:28:07]

This clearly breaks something :)

[yurys, 2015/07/02 21:21:26]

All the tests are passing and it works fine on usual scenarios. What breakages
do you mean?

On 2015/07/02 20:28:07, pfeldman wrote:

+            if (this.target().isPage() || this.target().isServiceWorker())
+                console.error(JSON.stringify(error));
+            this._cachedResourcesProcessed = true;
             return;
         }