[DevTools] Improved console.copy command to handle JSON.stringifyable objects.

LayoutTests/inspector/console/console-command-copy.html

Index: LayoutTests/inspector/console/console-command-copy.html
diff --git a/LayoutTests/inspector/console/console-command-copy.html b/LayoutTests/inspector/console/console-command-copy.html
index 6e21c2a9e3ef883914355d7d82ef964939f6c97d..fc116c4c95f5186a39aa04fb1c296d1fce009e84 100644
--- a/LayoutTests/inspector/console/console-command-copy.html
+++ b/LayoutTests/inspector/console/console-command-copy.html
@@ -6,12 +6,27 @@
 
 function test()
 {
-    InspectorTest.evaluateInConsole("copy('qwerty')");
+    var pendingCount = 9;
     InspectorFrontendHost.copyText = copyText;
-
+    
+    InspectorTest.evaluateInConsole("copy('qwerty')");
+    InspectorTest.evaluateInConsole("copy(document.querySelector('p'))");
+    InspectorTest.evaluateInConsole("copy({foo:'bar'})");
+    // Check cyclic references in objects
+    InspectorTest.evaluateInConsole("var a = {}; a.b = a; copy(a)");
+    InspectorTest.evaluateInConsole("copy(document.all)");
+    InspectorTest.evaluateInConsole("copy(NaN)");
+    InspectorTest.evaluateInConsole("copy(Infinity)");
+    InspectorTest.evaluateInConsole("copy(null)");
+    InspectorTest.evaluateInConsole("copy(undefined)");
+    InspectorTest.evaluateInConsole("copy(1)");
+    InspectorTest.evaluateInConsole("copy(true)");
+    InspectorTest.evaluateInConsole("copy(false)");

[aandrey, 2013/12/04 11:45:31]

could you also add smth like:

InspectorTest.evaluateInConsole("copy('string \"foo\" \\\\ \r\n')");

?

[Dmitry Zvorygin, 2013/12/04 16:00:40]

Done.

+    
     function copyText(text) {
         InspectorTest.addResult("InspectorFrontendHost.copyText: " + text);
-        InspectorTest.completeTest();
+        if (--pendingCount === 0)
+            InspectorTest.completeTest();

[pfeldman, 2013/12/04 11:48:56]

Just place this into the callback of evaluateInConsole, no need to count things.

[Dmitry Zvorygin, 2013/12/04 16:00:40]

Seems copy is kind of async operation and can be performed AFTER
evaluateInConsole callback, thus explicit counter is needed.

     }
 }