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

Source/core/inspector/InjectedScriptSource.js

 /*
  * Copyright (C) 2007 Apple Inc.  All rights reserved.
  * Copyright (C) 2013 Google Inc. 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 1365 matching lines @@
      * @param {*} object
      * @return {*}
      */
     inspect: function(object)
     {
         return injectedScript._inspect(object);
     },
 
     copy: function(object)
     {
-        if (injectedScript._subtype(object) === "node")
-            object = object.outerHTML;
-        var string = toString(object);
+        var string;
+        if (injectedScript._subtype(object) === "node") {
+            string = object.outerHTML;
+        } else if (typeof object === "string") {

[pfeldman, 2013/12/03 17:47:52]

Why special-casing the string?

[Dmitry Zvorygin, 2013/12/04 11:12:48]

JSON.stringify(str) would put "str" into quotes.

+            string = object;
+        } else {
+            try {
+                string = JSON.stringify(object, null, "  ");

[aandrey, 2013/12/03 17:49:42]

1) string can be undefined after this: JSON.stringify(undefined)
2) JSON.stringify(NaN) => "null". Same for Infinity, -Infinity.

[Dmitry Zvorygin, 2013/12/04 11:12:48]

Done.

+            } catch (e) {
+                string = toString(object);
+            }
+        }
+
         var hints = { copyToClipboard: true };
         var remoteObject = injectedScript._wrapObject(string, "")
         InjectedScriptHost.inspect(remoteObject, hints);
     },
 
     clear: function()
     {
         InjectedScriptHost.clearConsoleMessages();
     },
 
@@ skipping 85 matching lines @@
      */
     _logEvent: function(event)
     {
         inspectedWindow.console.log(event.type, event);
     }
 }
 
 injectedScript._commandLineAPIImpl = new CommandLineAPIImpl();
 return injectedScript;
 })