Bug 13088

Summary: REGRESSION(r19761-19779): Copy image no longer includes image address
Product: WebKit Reporter: Denis Ahrens <denis>
Component: WebCore Misc.Assignee: Beth Dakin <bdakin>
Severity: Normal CC: bdakin, dev+webkit
Priority: P1 Keywords: InRadar, Regression
Version: 523.x (Safari 3)   
Hardware: Mac (Intel)   
OS: OS X 10.4   
Description Flags
Declare types only once
Actually this one is better bdakin: review+

Description Denis Ahrens 2007-03-15 13:49:19 PDT
When I copy an image to the clipboard with webkit the image is there but not the URL. When I do the same in Safari there is the image AND the URL. This is a REGRESSION!
Comment 1 Matt Lilek 2007-03-15 14:26:34 PDT
1. Load http://webkit.org/images/icon-gold.png
2. In ToT, right click and choose "Copy Image"
3. Open a new tab/window and paste into the address bar - nothing
4. Repeat in Safari 2.0.4 - the image address is pasted fine.

Viewing the clipboard with Clipboard Viewer (in the Developer Tools) shows a bunch of differences between what's copied from ToT and Safari 2.0.4.  Both have identical contents for NeXT TIFF v4.0 pasteboard type.  ToT includes a NeXT RTFD pasteboard type that shipping Safari does not.  For everything else (WebURLsWithTitlesPboardType, Apple URL pasteboard type, 'url', 'urln', NSStringPboardType, NeXT plain ascii pasteboard type, Apple PICT pasteboard type), ToT has nothing while shipping Safari fills all those in.
Comment 2 Denis Ahrens 2007-03-15 19:08:54 PDT
in Webkit r19761 it works like in Safari but in Webkit r19779 it does not.
Comment 3 Matt Lilek 2007-03-15 19:18:48 PDT
Thanks for regressing that Denis!

http://trac.webkit.org/projects/webkit/changeset/19764 looks to me to be the culprit.
Comment 4 Mark Rowe (bdash) 2007-03-15 22:21:34 PDT
Comment 5 Beth Dakin 2007-03-15 23:27:28 PDT
Created attachment 13655 [details]
Declare types only once
Comment 6 Beth Dakin 2007-03-15 23:46:34 PDT
Created attachment 13656 [details]
Actually this one is better
Comment 7 Beth Dakin 2007-03-15 23:51:36 PDT
Comment on attachment 13656 [details]
Actually this one is better

Geoff reviewed this.
Comment 8 Beth Dakin 2007-03-16 00:05:16 PDT
Fixed in r20233.