[Storage] Cache storage inspection on all the frames!

Source/devtools/front_end/resources/ResourcesPanel.js

 /*
  * Copyright (C) 2007, 2008, 2010 Apple Inc.  All rights reserved.
  * Copyright (C) 2009 Joseph Pecoraro
  * 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
@@ skipping 48 matching lines @@
 
     this.sessionStorageListTreeElement = new WebInspector.StorageCategoryTreeElement(this, WebInspector.UIString("Session Storage"), "SessionStorage", ["domstorage-storage-tree-item", "session-storage"]);
     this._sidebarTree.appendChild(this.sessionStorageListTreeElement);
 
     this.cookieListTreeElement = new WebInspector.StorageCategoryTreeElement(this, WebInspector.UIString("Cookies"), "Cookies", ["cookie-storage-tree-item"]);
     this._sidebarTree.appendChild(this.cookieListTreeElement);
 
     this.applicationCacheListTreeElement = new WebInspector.StorageCategoryTreeElement(this, WebInspector.UIString("Application Cache"), "ApplicationCache", ["application-cache-storage-tree-item"]);
     this._sidebarTree.appendChild(this.applicationCacheListTreeElement);
 
+    this.serviceWorkerCacheListTreeElement = new WebInspector.ServiceWorkerCacheTreeElement(this);
+    this._sidebarTree.appendChild(this.serviceWorkerCacheListTreeElement);
+
     if (Runtime.experiments.isEnabled("fileSystemInspection")) {
         this.fileSystemListTreeElement = new WebInspector.FileSystemListTreeElement(this);
         this._sidebarTree.appendChild(this.fileSystemListTreeElement);
     }
 
     var mainView = new WebInspector.VBox();
     this.storageViews = mainView.element.createChild("div", "vbox flex-auto");
     this._storageViewStatusBar = new WebInspector.StatusBar(mainView.element);
     this._storageViewStatusBar.element.classList.add("resources-status-bar");
     this.splitView().setMainView(mainView);
@@ skipping 36 matching lines @@
     /**
      * @override
      * @param {!WebInspector.Target} target
      */
     targetAdded: function(target)
     {
         if (this._target)
             return;
         this._target = target;
 
-        if (target.isServiceWorker()) {
-            this.serviceWorkerCacheListTreeElement = new WebInspector.ServiceWorkerCacheTreeElement(this);
-            this._sidebarTree.appendChild(this.serviceWorkerCacheListTreeElement);
-        }
-
         if (target.serviceWorkerManager && Runtime.experiments.isEnabled("serviceWorkersInResources")) {
             this.serviceWorkersTreeElement = new WebInspector.ServiceWorkersTreeElement(this);
             this._sidebarTree.appendChild(this.serviceWorkersTreeElement);
         }
 
         this._databaseModel = WebInspector.DatabaseModel.fromTarget(target);
         this._domStorageModel = WebInspector.DOMStorageModel.fromTarget(target);
 
         if (target.resourceTreeModel.cachedResourcesLoaded())
             this._initialize();
@@ skipping 20 matching lines @@
         this._databaseModel.removeEventListener(WebInspector.DatabaseModel.Events.DatabaseAdded, this._databaseAdded, this);
 
         this._resetWithFrames();
     },
 
     _initialize: function()
     {
         this._databaseModel.enable();
         this._domStorageModel.enable();
         WebInspector.IndexedDBModel.fromTarget(this._target).enable();
+        WebInspector.ServiceWorkerCacheModel.fromTarget(this._target).enable();
 
         if (this._target.isPage())
             this._populateResourceTree();
         this._populateDOMStorageTree();
         this._populateApplicationCacheTree();
         this.indexedDBListTreeElement._initialize();
         if (this.serviceWorkerCacheListTreeElement)
             this.serviceWorkerCacheListTreeElement._initialize();
         if (Runtime.experiments.isEnabled("fileSystemInspection"))
             this.fileSystemListTreeElement._initialize();
@@ skipping 1235 matching lines @@
     },
 
     __proto__: WebInspector.BaseStorageTreeElement.prototype
 }
 
 
 /**
  * @constructor
  * @extends {WebInspector.StorageCategoryTreeElement}
  * @param {!WebInspector.ResourcesPanel} storagePanel
- * @implements {WebInspector.TargetManager.Observer}
  */
 WebInspector.ServiceWorkerCacheTreeElement = function(storagePanel)
 {
-    WebInspector.StorageCategoryTreeElement.call(this, storagePanel, WebInspector.UIString("Service Worker Cache"), "ServiceWorkerCache", ["service-worker-cache-storage-tree-item"]);
+    WebInspector.StorageCategoryTreeElement.call(this, storagePanel, WebInspector.UIString("Cache Storage"), "Cache Storage", ["service-worker-cache-storage-tree-item"]);

[pfeldman, 2015/04/01 07:01:09]

"Cache Storage" -> "CacheStorage"

[dmurph, 2015/04/02 20:59:02]

Done.

 }
 
 WebInspector.ServiceWorkerCacheTreeElement.prototype = {
     _initialize: function()
     {
         /** @type {!Array.<!WebInspector.SWCacheTreeElement>} */
         this._swCacheTreeElements = [];
         var targets = WebInspector.targetManager.targets();
-        for (var i = 0; i < targets.length; ++i) {
-            if (!targets[i].serviceWorkerCacheModel)
-                continue;
-            var caches = targets[i].serviceWorkerCacheModel.caches();
-            for (var j = 0; j < caches.length; ++j)
-                this._addCache(targets[i].serviceWorkerCacheModel, caches[j]);
+        for (var target of targets) {

[pfeldman, 2015/04/01 07:01:09]

Targets can be added later, you should keep the target observer. Also, note that
when the Page is a main target, the SW target can be added at a random moment
and will belong to the same origin. So you need to make sure you don't dupe
cache storages. Make sure to test it on the SW-controlled pages. It might be
that you only care about the first target in case of Cache Storage.

[dmurph, 2015/04/02 20:59:02]

I don't need to because we listen for securityOrigins on the model, which get
called when targets are added (unless I'm wrong).

The cache names are treated as a set, before the 'cache added' or 'removed'
events are sent out, so there shouldn't be duplicates.

+            var model = WebInspector.ServiceWorkerCacheModel.fromTarget(target);
+            var caches = model.caches();
+            for (var cache of caches)
+                this._addCache(model, cache);
         }
         WebInspector.targetManager.addModelListener(WebInspector.ServiceWorkerCacheModel, WebInspector.ServiceWorkerCacheModel.EventTypes.CacheAdded, this._cacheAdded, this);
         WebInspector.targetManager.addModelListener(WebInspector.ServiceWorkerCacheModel, WebInspector.ServiceWorkerCacheModel.EventTypes.CacheRemoved, this._cacheRemoved, this);
-        this._refreshCaches();
-        WebInspector.targetManager.observeTargets(this);
     },
 
     onattach: function()
     {
         WebInspector.StorageCategoryTreeElement.prototype.onattach.call(this);
         this.listItemElement.addEventListener("contextmenu", this._handleContextMenuEvent.bind(this), true);
     },
 
-    /**
-     * @override
-     * @param {!WebInspector.Target} target
-     */
-    targetAdded: function(target)
-    {
-        if (target.isServiceWorker() && target.serviceWorkerCacheModel)
-            this._refreshCaches();
-    },
-
-    /**
-     * @override
-     * @param {!WebInspector.Target} target
-     */
-    targetRemoved: function(target) {},
-
     _handleContextMenuEvent: function(event)
     {
         var contextMenu = new WebInspector.ContextMenu(event);
         contextMenu.appendItem(WebInspector.UIString("Refresh Caches"), this._refreshCaches.bind(this));
         contextMenu.show();
     },
 
     _refreshCaches: function()
     {
         var targets = WebInspector.targetManager.targets();
-        for (var i = 0; i < targets.length; ++i) {
-            if (targets[i].serviceWorkerCacheModel)
-                targets[i].serviceWorkerCacheModel.refreshCacheNames();
-        }
+        for (var target of targets)
+            WebInspector.ServiceWorkerCacheModel.fromTarget(target).refreshCacheNames();
     },
 
     /**
      * @param {!WebInspector.Event} event
      */
     _cacheAdded: function(event)
     {
         var cacheId = /** @type {!WebInspector.ServiceWorkerCacheModel.CacheId} */ (event.data);
         var model = /** @type {!WebInspector.ServiceWorkerCacheModel} */ (event.target);
         this._addCache(model, cacheId);
@@ skipping 51 matching lines @@
 
 /**
  * @constructor
  * @extends {WebInspector.BaseStorageTreeElement}
  * @param {!WebInspector.ResourcesPanel} storagePanel
  * @param {!WebInspector.ServiceWorkerCacheModel} model
  * @param {!WebInspector.ServiceWorkerCacheModel.CacheId} cacheId
  */
 WebInspector.SWCacheTreeElement = function(storagePanel, model, cacheId)
 {
-    WebInspector.BaseStorageTreeElement.call(this, storagePanel, cacheId.name, ["service-worker-cache-tree-item"]);
+    WebInspector.BaseStorageTreeElement.call(this, storagePanel, cacheId.name + " - " + cacheId.securityOrigin, ["service-worker-cache-tree-item"]);
     this._model = model;
     this._cacheId = cacheId;
 }
 
 WebInspector.SWCacheTreeElement.prototype = {
     get itemURL()
     {
         // I don't think this will work at all.
         return "swcache://" + this._cacheId.name;
     },
@@ skipping 863 matching lines @@
 WebInspector.ResourcesPanelFactory.prototype = {
     /**
      * @override
      * @return {!WebInspector.Panel}
      */
     createPanel: function()
     {
         return WebInspector.ResourcesPanel._instance();
     }
 }