[DevTools] Make resource tree model optional

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

 /*
  * Copyright (C) 2008 Nokia Inc.  All rights reserved.
  * Copyright (C) 2013 Samsung Electronics. 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 114 matching lines @@
     DOMStorageRemoved: "DOMStorageRemoved"
 }
 
 WebInspector.DOMStorageModel.prototype = {
     enable: function()
     {
         if (this._enabled)
             return;
 
         this.target().registerDOMStorageDispatcher(new WebInspector.DOMStorageDispatcher(this));
-        this.target().resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.SecurityOriginAdded, this._securityOriginAdded, this);
-        this.target().resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.SecurityOriginRemoved, this._securityOriginRemoved, this);
+        var resourceTreeModel = WebInspector.ResourceTreeModel.fromTarget(this.target());
+        if (resourceTreeModel) {
+            resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.SecurityOriginAdded, this._securityOriginAdded, this);
+            resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.SecurityOriginRemoved, this._securityOriginRemoved, this);
+
+            var securityOrigins = resourceTreeModel.securityOrigins();

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

Looks like this model doesn't function without security origins. Let's make it
explicitly depend on ResourceTreeModel instead.

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

Done.

+            for (var i = 0; i < securityOrigins.length; ++i)
+                this._addOrigin(securityOrigins[i]);
+        }
         this._agent.enable();
 
-        var securityOrigins = this.target().resourceTreeModel.securityOrigins();
-        for (var i = 0; i < securityOrigins.length; ++i)
-            this._addOrigin(securityOrigins[i]);
-
         this._enabled = true;
     },
 
     /**
      * @param {string} origin
      */
     clearForOrigin: function(origin)
     {
         if (!this._enabled)
             return;
@@ skipping 204 matching lines @@
  * @param {!WebInspector.Target} target
  * @return {!WebInspector.DOMStorageModel}
  */
 WebInspector.DOMStorageModel.fromTarget = function(target)
 {
     if (!target[WebInspector.DOMStorageModel._symbol])
         target[WebInspector.DOMStorageModel._symbol] = new WebInspector.DOMStorageModel(target);
 
     return target[WebInspector.DOMStorageModel._symbol];
 }