Attachment 75057 Details for Bug 50180 – [PATCH] Proposed change

[patch] [PATCH] Proposed change

copy_node (text/plain), 1.26 KB, created by Pavel Feldman on 2010-11-29 13:58:36 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Pavel Feldman

Created: 2010-11-29 13:58:36 PST

Size: 1.26 KB

patch

obsolete

>diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
>index 991a09d..3f549c4 100644
>--- a/WebCore/ChangeLog
>+++ b/WebCore/ChangeLog
>@@ -1,3 +1,13 @@
>+2010-11-29  Pavel Feldman  <pfeldman@chromium.org>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Web Inspector: make copy(node) work in console.
>+        https://bugs.webkit.org/show_bug.cgi?id=50180
>+
>+        * inspector/front-end/InjectedScript.js:
>+        (injectedScriptConstructor.):
>+
> 2010-11-29  Patrick Gansterer  <paroga@webkit.org>
> 
>         Reviewed by Adam Roben.
>diff --git a/WebCore/inspector/front-end/InjectedScript.js b/WebCore/inspector/front-end/InjectedScript.js
>index 106668f..fb6b796 100644
>--- a/WebCore/inspector/front-end/InjectedScript.js
>+++ b/WebCore/inspector/front-end/InjectedScript.js
>@@ -700,11 +700,9 @@ CommandLineAPI.prototype = {
> 
>     copy: function(object)
>     {
>-        if (injectedScript._type(object) === "node") {
>-            var nodeId = InjectedScriptHost.pushNodePathToFrontend(object, false, false);
>-            InjectedScriptHost.copyNode(nodeId);
>-        } else
>-            InjectedScriptHost.copyText(object);
>+        if (injectedScript._type(object) === "node")
>+            object = object.outerHTML;
>+        InjectedScriptHost.copyText(object);
>     },
> 
>     clear: function()

Actions: View | Formatted Diff | Diff

Attachments on bug 50180: 75057