[DevTools] Make resource tree model optional

third_party/WebKit/Source/devtools/front_end/resources/AppManifestView.js

 // Copyright (c) 2016 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.VBox}
  * @implements {WebInspector.TargetManager.Observer}
  */
 WebInspector.AppManifestView = function()
@@ skipping 52 matching lines @@
     /**
      * @override
      * @param {!WebInspector.Target} target
      */
     targetRemoved: function(target)
     {
     },
 
     _updateManifest: function()
     {
-        this._target.resourceTreeModel.fetchAppManifest(this._renderManifest.bind(this));
+        var resourceTreeModel = WebInspector.ResourceTreeModel.fromTarget(this._target);

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

I'd rather set this._resourceTreeModel in targetAdded instead of this._target.
This view doesn't work without ResourceTreeModel.

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

Done.

+        if (resourceTreeModel)
+            resourceTreeModel.fetchAppManifest(this._renderManifest.bind(this));
     },
 
     /**
      * @param {string} url
      * @param {?string} data
      * @param {!Array<!PageAgent.AppManifestError>} errors
      */
     _renderManifest: function(url, data, errors)
     {
         this._reportView.setURL(url);
@@ skipping 49 matching lines @@
     {
         var target = WebInspector.targetManager.mainTarget();
         if (target && target.hasBrowserCapability()) {
             target.pageAgent().requestAppBanner();
             WebInspector.console.show();
         }
     },
 
     __proto__: WebInspector.VBox.prototype
 }