[DevTools] Make resource tree model optional

third_party/WebKit/Source/devtools/front_end/sdk/WorkerManager.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 39 matching lines @@
 }
 
 WebInspector.WorkerManager.prototype = {
     enable: function()
     {
         if (this._enabled)
             return;
         this._enabled = true;
 
         this.target().workerAgent().enable();
-        this.target().resourceTreeModel.addEventListener(WebInspector.TargetManager.Events.MainFrameNavigated, this._mainFrameNavigated, this);
+        var resourceTreeModel = WebInspector.ResourceTreeModel.fromTarget(this.target());
+        if (resourceTreeModel)
+            resourceTreeModel.addEventListener(WebInspector.TargetManager.Events.MainFrameNavigated, this._mainFrameNavigated, this);
     },
 
     disable: function()
     {
         if (!this._enabled)
             return;
         this._enabled = false;
         this._reset();
         this.target().workerAgent().disable();
-        this.target().resourceTreeModel.removeEventListener(WebInspector.TargetManager.Events.MainFrameNavigated, this._mainFrameNavigated, this);
+        var resourceTreeModel = WebInspector.ResourceTreeModel.fromTarget(this.target());
+        if (resourceTreeModel)
+            resourceTreeModel.removeEventListener(WebInspector.TargetManager.Events.MainFrameNavigated, this._mainFrameNavigated, this);

[dgozman, 2016/07/14 16:29:29]

This is a very strange dependency. I think it should no be necessary, but let's
dig up why it was added.

[eostroukhov-old, 2016/07/20 23:46:16]

Switched to listening to TargetManager instead.

     },
 
     dispose: function()
     {
         this._reset();
     },
 
     _reset: function()
     {
         for (var connection of this._connections.values())
@@ skipping 140 matching lines @@
         this._agent.sendMessageToWorker(this._workerId, JSON.stringify(messageObject));
     },
 
     _close: function()
     {
         this.connectionClosed("worker_terminated");
     },
 
     __proto__: InspectorBackendClass.Connection.prototype
 }