[DevTools] Make resource tree model optional

third_party/WebKit/Source/devtools/front_end/animation/AnimationModel.js

 // Copyright (c) 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 
 /**
  * @constructor
  * @extends {WebInspector.SDKModel}
  * @param {!WebInspector.Target} target
  */
 WebInspector.AnimationModel = function(target)
 {
     WebInspector.SDKModel.call(this, WebInspector.AnimationModel, target);
     this._agent = target.animationAgent();
     target.registerAnimationDispatcher(new WebInspector.AnimationDispatcher(this));
     /** @type {!Map.<string, !WebInspector.AnimationModel.Animation>} */
     this._animationsById = new Map();
     /** @type {!Map.<string, !WebInspector.AnimationModel.AnimationGroup>} */
     this._animationGroups = new Map();
     /** @type {!Array.<string>} */
     this._pendingAnimations = [];
     this._playbackRate = 1;
-    target.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.MainFrameNavigated, this._reset, this);
+    var resourceTreeModel = WebInspector.ResourceTreeModel.fromTarget(target);
+    if (resourceTreeModel)

[pfeldman, 2016/07/07 17:44:13]

No need in this check - we should either have both resourcetreemodel and
animation, or none.

[eostroukhov-old, 2016/07/13 23:30:58]

To signify the dependency, I turned resourceTreeModel into ctor parameter.

+        resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.MainFrameNavigated, this._reset, this);
     this._screenshotCapture = new WebInspector.AnimationModel.ScreenshotCapture(target, this);
 }
 
 WebInspector.AnimationModel.Events = {
     AnimationGroupStarted: "AnimationGroupStarted",
     ModelReset: "ModelReset"
 }
 
 WebInspector.AnimationModel.prototype = {
     _reset: function()
@@ skipping 842 matching lines @@
 /**
  * @constructor
  * @param {!WebInspector.Target} target
  * @param {!WebInspector.AnimationModel} model
  */
 WebInspector.AnimationModel.ScreenshotCapture = function(target, model)
 {
     this._target = target;
     /** @type {!Array<!WebInspector.AnimationModel.ScreenshotCapture.Request>} */
     this._requests = [];
-    this._target.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.ScreencastFrame, this._screencastFrame, this);
+    var resourceTreeModel = WebInspector.ResourceTreeModel.fromTarget(this._target);
+    if (resourceTreeModel)

[pfeldman, 2016/07/07 17:44:13]

ditto

[eostroukhov-old, 2016/07/13 23:30:58]

Done.

+        resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.ScreencastFrame, this._screencastFrame, this);
     this._model = model;
     this._model.addEventListener(WebInspector.AnimationModel.Events.ModelReset, this._stopScreencast, this);
 }
 
 /** @typedef {{ endTime: number, screenshots: !Array.<string>}} */
 WebInspector.AnimationModel.ScreenshotCapture.Request;
 
 WebInspector.AnimationModel.ScreenshotCapture.prototype = {
     /**
      * @param {number} duration
@@ skipping 46 matching lines @@
         if (!this._capturing)
             return;
 
         delete this._stopTimer;
         delete this._endTime;
         this._requests = [];
         this._capturing = false;
         this._target.pageAgent().stopScreencast();
     }
 }