|Summary:||[chromium] Add new WebClipboard method for setting generic data in copy/cut handlers.|
|Product:||WebKit||Reporter:||Daniel Cheng <dcheng>|
|Severity:||Normal||CC:||commit-queue, fishd, tony|
|Version:||528+ (Nightly build)|
|OS:||OS X 10.5|
|Bug Depends on:|
Description Daniel Cheng 2010-07-12 17:40:14 PDT
Comment 1 Daniel Cheng 2010-07-12 17:43:27 PDT
Created attachment 61301 [details] Patch
Comment 2 Darin Fisher (:fishd, Google) 2010-07-12 22:41:01 PDT
Comment on attachment 61301 [details] Patch you need to update the ChangeLog file. it'd be helpful also if i could see how this new function will be used.
Comment 3 Daniel Cheng 2010-07-13 14:53:51 PDT
Created attachment 61419 [details] Patch Sorry, forgot to commit the changes for the ChangeLog. ClipboardChromium::setData (or ChromiumDataObject) will call ChromiumBridge::clipboard()->writeDataFlavor() directly when setting data for a copy/cut. As mentioned in bug 40515, it'd be nice if we had the equivalent of startDrag() for copy and cut, but I don't think that feature is going to make M6.
Comment 4 Tony Chang 2010-08-02 13:48:02 PDT
I think this would be easier to review if you had a chromium side change demonstrating how it's used.
Comment 5 Eric Seidel (no email) 2010-08-06 13:49:44 PDT
Comment on attachment 61419 [details] Patch r- based on tony and darin's comments above.
Comment 6 Daniel Cheng 2010-08-23 16:21:57 PDT
(In reply to comment #4) > I think this would be easier to review if you had a chromium side change demonstrating how it's used. I'll try to post a patch for that this evening. I'm trying to separate my patches from each other right now, and it's a rather painful process. It's needed because drags and copies/cuts aren't written back to the system data object the same way. For drags: The drag controller starts a drag. The DOM receives a dragstart event and populates event.dataTransfer. The drag controller calls DragClient::startDrag(), which passes all the data that was set in event.dataTransfer to the Chromium side. For copy/cut: The editor clears the pasteboard before it does anything. Then a copy/cut event is dispatched to the DOM, and for each call to event.dataTransfer.setData(), data is immediately written back to the system pasteboard. There is no analogous version of DragClient::startDrag() for pasteboard operations. I wanted to add this, but haven't done so since I didn't receive feedback from Darin Adler. Since we must write back immediately, and the existing APIs are not rich enough, I added a new interface.
Comment 7 Daniel Cheng 2010-08-26 15:32:23 PDT
Created attachment 65628 [details] Patch
Comment 8 Tony Chang 2010-08-26 16:19:39 PDT
Comment on attachment 65628 [details] Patch I see, we're not using writeData yet (it looks like a stub on the chromium side). In that case, this seems fine. Don't forget to update webclipboard_impl.* on the Chromium side too.
Comment 9 WebKit Commit Bot 2010-08-27 19:25:44 PDT
Comment on attachment 65628 [details] Patch Clearing flags on attachment: 65628 Committed r66293: <http://trac.webkit.org/changeset/66293>
Comment 10 WebKit Commit Bot 2010-08-27 19:25:50 PDT
All reviewed patches have been landed. Closing bug.