[Storage] Cache storage inspection on all the frames!

LayoutTests/http/tests/inspector/inspector-test.js

 var initialize_InspectorTest = function() {
 
 var results = [];
 
 function consoleOutputHook(messageType)
 {
     InspectorTest.addResult(messageType + ": " + Array.prototype.slice.call(arguments, 1));
 }
 
 window._originalConsoleLog = console.log.bind(console);
@@ skipping 40 matching lines @@
 
 InspectorTest.evaluateInPageWithTimeout = function(code)
 {
     // FIXME: we need a better way of waiting for chromium events to happen
     InspectorTest.evaluateInPage("setTimeout(unescape('" + escape(code) + "'), 1)");
 }
 
 var lastEvalId = 0;
 var pendingEvalRequests = {};
 
+var lastPromiseEvalId = 0;
+var pendingPromiseEvalRequests = {};
+
+/**
+ * The given function should take two callback paraters before the arguments:
+ *  * resolve - called when successful (with optional result)
+ *  * reject  - called when there was a failure (with optional error)
+ */
+InspectorTest.invokePageFunctionPromise = function(functionName, parameters)
+{
+    return new Promise(function(resolve, reject) {
+        var id = ++lastPromiseEvalId;
+        pendingPromiseEvalRequests[id] = { resolve: InspectorTest.safeWrap(resolve), reject: InspectorTest.safeWrap(reject) };
+
+        var jsonParameters = [];
+        for (var i = 0; i < parameters.length; ++i)
+            jsonParameters.push(JSON.stringify(parameters[i]));
+        var asyncEvalWrapper = function(callId, functionName, argumentsArray)
+        {
+            function evalCallbackResolve(result)
+            {
+                testRunner.evaluateInWebInspector(evalCallbackCallId, "InspectorTest.didInvokePageFunctionPromise(" + callId + ", " + JSON.stringify(result) + ", true);");
+            }
+            function evalCallbackReject(result)

[pfeldman, 2015/04/15 17:50:08]

Blank lines around functions.

[dmurph, 2015/04/15 21:29:32]

Done.

+            {
+                testRunner.evaluateInWebInspector(evalCallbackCallId, "InspectorTest.didInvokePageFunctionPromise(" + callId + ", " + JSON.stringify(result) + ", false);");
+            }
+            var args = [evalCallbackResolve, evalCallbackReject].concat(argumentsArray.map(JSON.stringify));
+            var functionCall = functionName + ".call(null, " + args.join(", ") + ")";
+            try {
+                eval(functionCall);
+            } catch(e) {
+                console.error(e);

[pfeldman, 2015/04/15 17:50:08]

Inspector.addResult, no console.error please, they are not what you think (see
the beginning of this file)

[dmurph, 2015/04/15 21:29:32]

Done. (This was copied from below, so I changed invokePageFunctionAsync as well)

+                evalCallbackReject(e);
+            }
+        }
+        var pageRequest = "(" + asyncEvalWrapper.toString() + ")(" + id + ", unescape('" + escape(functionName) + "'), [" + jsonParameters.join(", ") + "])";
+        InspectorTest.evaluateInPage(pageRequest);
+    });
+}
+
+
+InspectorTest.didInvokePageFunctionPromise = function(callId, value, didResolve)
+{
+    var callbacks = pendingPromiseEvalRequests[callId];
+    if (!callbacks) {
+        InspectorTest.addResult("Missing callback for async eval " + callId + ", perhaps callback invoked twice?");
+        return;
+    }
+    var callback = didResolve ? callbacks.resolve : callbacks.reject;
+    delete pendingPromiseEvalRequests[callId];
+    callback(value);
+}
+
 InspectorTest.invokePageFunctionAsync = function(functionName, callback)
 {
     var id = ++lastEvalId;
     pendingEvalRequests[id] = InspectorTest.safeWrap(callback);
     var asyncEvalWrapper = function(callId, functionName)
     {
         function evalCallback(result)
         {
             testRunner.evaluateInWebInspector(evalCallbackCallId, "InspectorTest.didInvokePageFunctionAsync(" + callId + ", " + JSON.stringify(result) + ");");
         }
@@ skipping 351 matching lines @@
         try {
             Array.prototype.push.call(arguments, result);
             override.apply(this, arguments);
         } catch (e) {
             throw ("Exception in overriden method '" + methodName + "': " + e);
         }
         return result;
     };
 }
 
+InspectorTest.addSnifferPromise = function(receiver, methodName)
+{
+    return new Promise(function (resolve, reject) {
+            var original = receiver[methodName];

[pfeldman, 2015/04/15 17:50:08]

Poor indent.

[dmurph, 2015/04/15 21:29:32]

Done.

+            if (typeof original !== "function") {
+                reject("Cannot find method to override: " + methodName);
+                return;
+            }
+
+            receiver[methodName] = function(var_args) {
+                try {
+                    var result = original.apply(this, arguments);
+                } finally {
+                    receiver[methodName] = original;
+                }
+                // In case of exception the override won't be called.
+                try {
+                    Array.prototype.push.call(arguments, result);
+                    resolve.apply(this, arguments);
+                } catch (e) {
+                    reject("Exception in overriden method '" + methodName + "': " + e);
+                }
+                return result;
+            };
+        });
+}
+
 InspectorTest.addConsoleSniffer = function(override, opt_sticky)
 {
     InspectorTest.addSniffer(WebInspector.ConsoleModel.prototype, "addMessage", override, opt_sticky);
 }
 
 InspectorTest.override = function(receiver, methodName, override, opt_sticky)
 {
     override = InspectorTest.safeWrap(override);
 
     var original = receiver[methodName];
@@ skipping 532 matching lines @@
     _output("[page] " + text);
 }
 
 function _output(result)
 {
     if (!outputElement)
         createOutputElement();
     outputElement.appendChild(document.createTextNode(result));
     outputElement.appendChild(document.createElement("br"));
 }