Bug 76464 - Web Inspector: CopyText is not implemented for remote web inspector
Summary: Web Inspector: CopyText is not implemented for remote web inspector
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-17 10:15 PST by Konrad Piascik
Modified: 2014-12-12 14:35 PST (History)
10 users (show)

See Also:


Attachments
patch (2.27 KB, patch)
2012-01-20 08:01 PST, Konrad Piascik
pfeldman: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Konrad Piascik 2012-01-17 10:15:50 PST
The soft context menu options:
Copy Link Address
Copy Request Headers
Copy Response Headers
Copy Entry as HAR
Copy All as HAR

do not function at all since InspectorFrontendHostStub.js has an empty implementation for the copyText function.
Comment 1 Konrad Piascik 2012-01-20 08:01:21 PST
Created attachment 123317 [details]
patch

partial patch to have all copy's use the same copy method.
Comment 2 Pavel Feldman 2012-01-20 08:18:17 PST
Comment on attachment 123317 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=123317&action=review

You should also remove copyNode from the Inspector.json and InspectorDOMAgent.h/cpp.

> Source/WebCore/inspector/front-end/ElementsTreeOutline.js:1687
> +        var copy = function(id, text) {

Should be:

function copy(error, text)
{
    if (!error)
        ....
}
Comment 3 Pavel Feldman 2012-01-20 08:20:29 PST
You should also create a dedicated bug for this patch. As for the original issue, it can be solved via browser extensions. Chrome exposes access to the clipboard given the necessary permission is granted by the user.

Web platform does not and will not allow accessing clipboard from within the page at a random point of time.
Comment 4 Konrad Piascik 2012-11-20 07:35:59 PST
Comment on attachment 123317 [details]
patch

clear flag.
Comment 5 Brian Burg 2014-12-12 14:35:43 PST
Closing as invalid, as this bug pertains to the old inspector UI and/or its tests.
Please file a new bug (https://www.webkit.org/new-inspector-bug) if the bug/feature/issue is still relevant to WebKit trunk.