[DevTools] Explicitly require ResourceTreeModel

third_party/WebKit/Source/devtools/front_end/sdk/ApplicationCacheModel.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:
  *
  * 1. Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *
@@ skipping 21 matching lines @@
  * @param {!WebInspector.Target} target
  */
 WebInspector.ApplicationCacheModel = function(target)
 {
     WebInspector.SDKModel.call(this, WebInspector.ApplicationCacheModel, target);
 
     target.registerApplicationCacheDispatcher(new WebInspector.ApplicationCacheDispatcher(this));
     this._agent = target.applicationCacheAgent();
     this._agent.enable();
 
-    target.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.FrameNavigated, this._frameNavigated, this);
-    target.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.FrameDetached, this._frameDetached, this);
+    var resourceTreeModel = WebInspector.ResourceTreeModel.fromTarget(target);

[pfeldman, 2016/07/22 17:33:47]

There should be no AppCacheModel when there is no resource tree model.

[eostroukhov-old, 2016/07/25 19:36:10]

Done.

+    if (resourceTreeModel) {

[dgozman, 2016/07/22 17:19:43]

This model doesn't work without frames at all.

[eostroukhov-old, 2016/07/25 19:36:10]

Done.

+        resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.FrameNavigated, this._frameNavigated, this);
+        resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.FrameDetached, this._frameDetached, this);
+    }
 
     this._statuses = {};
     this._manifestURLsByFrame = {};
 
     this._mainFrameNavigated();
     this._onLine = true;
 }
 
 WebInspector.ApplicationCacheModel.EventTypes = {
     FrameManifestStatusUpdated: "FrameManifestStatusUpdated",
@@ skipping 215 matching lines @@
 }
 
 /**
  * @param {!WebInspector.Target} target
  * @return {?WebInspector.ApplicationCacheModel}
  */
 WebInspector.ApplicationCacheModel.fromTarget = function(target)
 {
     return /** @type {?WebInspector.ApplicationCacheModel} */ (target.model(WebInspector.ApplicationCacheModel));
 }