[Storage] Cache storage inspection on all the frames!

Source/devtools/protocol.json

 {
     "version": { "major": "1", "minor": "1" },
     "domains": [{
         "domain": "Inspector",
         "hidden": true,
         "types": [],
         "commands": [
             {
                 "name": "enable",
                 "description": "Enables inspector domain notifications."
@@ skipping 1663 matching lines @@
                     { "name": "databaseName", "type": "string", "description": "Database name." },
                     { "name": "objectStoreName", "type": "string", "description": "Object store name." }
                 ],
                 "returns": [
                 ],
                 "description": "Clears all entries from an object store."
             }
         ]
     },
     {
-        "domain": "ServiceWorkerCache",
+        "domain": "CacheStorage",
         "hidden": true,
         "types": [
             {
                 "id": "DataEntry",
                 "type": "object",
                 "description": "Data entry.",
                 "properties": [
                     { "name": "request", "type": "string", "description": "JSON-stringified request object." },
                     { "name": "response", "type": "string", "description": "JSON-stringified response object." }
                 ]
             }
         ],
         "commands": [
             {
                 "name": "requestCacheNames",
                 "async": true,
+                "parameters": [
+                    { "name": "securityOrigin", "type": "string", "description": "Security origin." }

[pfeldman, 2015/04/09 10:31:12]

Now sure why fetching by origin is important, lets fetch all.

+                ],
                 "returns": [
                     { "name": "cacheNames", "type": "array", "items": { "type": "string" }, "description": "Cache names for origin." }
                 ],
                 "description": "Requests cache names."
             },
             {
                 "name": "requestEntries",
                 "async": true,
                 "parameters": [
+                    { "name": "securityOrigin", "type": "string", "description": "Security origin." },

[pfeldman, 2015/04/03 09:15:58]

Composite ids is hard to work with for the clients, I'd rather expose cacheId
instead of a cache name + sequrity origin that would uniquely identify it. Even
at the expense of parseable composite "securityorigin:name" id at the backend
that is opaque to the front-end.

[dmurph, 2015/04/08 19:35:35]

Just to clarify, you want me to remove the securityOrigin and cacheName part and
instead use a cacheId string that would then be parsed in the backend?

Just FYI, IndexedDB does it this way.  Also, there are cases, like
requestCacheNames, where we only need the security origin.

Also, how is this hard to work with?

[pfeldman, 2015/04/09 10:31:12]

Yes.
Too bad for IndexedDB. Why would we need requestCacheNames per security origin?
Why composite primary key is bad? Well, because when working with the remote
instance, you need a handle to that instance that would identify it. You operate
on this instance using this handle to refer to it, you associate updates to the
instances based on that handle. So there is inevitably a lookup from the handle
to the instance and back. That's why clients of your API will be concatenating
the sequrityOrigin and cacheName in one string in order to flatten it and use a
single map for a lookup. Or even worse, will use hierarchical maps.

                     { "name": "cacheName", "type": "string", "description": "Cache name." },
                     { "name": "skipCount", "type": "integer", "description": "Number of records to skip." },
                     { "name": "pageSize", "type": "integer", "description": "Number of records to fetch." }
                 ],
                 "returns": [
                     { "name": "cacheDataEntries", "type": "array", "items": { "$ref": "DataEntry" }, "description": "Array of object store data entries." },
                     { "name": "hasMore", "type": "boolean", "description": "If true, there are more entries to fetch in the given range." }
                 ],
                 "description": "Requests data from cache."
             },
             {
                 "name": "deleteCache",
                 "async": true,
                 "parameters": [
+                    { "name": "securityOrigin", "type": "string", "description": "Security origin." },
                     { "name": "cacheName", "type": "string", "description": "Cache name." }
                 ],
                 "description": "Deletes a cache."
             }
         ]
     },
     {
         "domain": "DOMStorage",
         "hidden": true,
         "description": "Query and modify DOM storage.",
@@ skipping 3335 matching lines @@
                 ],
                 "returns": [
                     { "name": "accessibilityNode", "$ref": "AXNode", "description": "The <code>Accessibility.AXNode</code> for this DOM node, if it exists.", "optional": true }
                 ],
                 "description": "Fetches the accessibility node for this DOM node, if it exists.",
                 "hidden": true
             }
         ]
     }]
 }