DevTools: support targets that don't have Resource agent

Source/devtools/front_end/sdk/ResourceTreeModel.js

 /*
  * Copyright (C) 2011 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 89 matching lines @@
             return result;
     }
     return null;
 }
 
 WebInspector.ResourceTreeModel.prototype = {
     _fetchResourceTree: function()
     {
         /** @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;
         }
 
         this.dispatchEventToListeners(WebInspector.ResourceTreeModel.EventTypes.WillLoadCachedResources);
         this._inspectedPageURL = mainFramePayload.frame.url;
 
         // Do not process SW resources.
         if (this.target().isPage())
             this._addFramesRecursively(null, mainFramePayload);
         else
@@ skipping 784 matching lines @@
     },
 
     /**
      * @override
      */
     interstitialHidden: function()
     {
         // Frontend is not interested in interstitials.
     }
 }