Bug 34293 - Add writeToPasteboard function to Clipboard interface and add stub implementations
Summary: Add writeToPasteboard function to Clipboard interface and add stub implementa...
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows Vista
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 34288
  Show dependency treegraph
 
Reported: 2010-01-28 16:57 PST by Daniel Cheng
Modified: 2010-02-04 15:36 PST (History)
0 users

See Also:


Attachments
Patch (9.55 KB, patch)
2010-01-28 17:00 PST, Daniel Cheng
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Cheng 2010-01-28 16:57:33 PST
This particular implementation was chosen to simplify potential Windows implementations of Clipboard::clearData(). It also helps with copy-and-paste as drag-and-drop; though that part of the spec is current in flux, it doesn't hurt keep it in mind when making patches in this area.

Calling writeToPasteboard() should do the following:
if (!isForDragging()) return false;
if (!isDirty()) return true;  // Dirty bit is tracked by each implementation.
/* copy data to system pasteboard */

Since the first two lines should be common to all implementations, should I split it into two functions, writeToPasteboard() and virtual writeToPasteboardImpl()?
Comment 1 Daniel Cheng 2010-01-28 17:00:02 PST
Created attachment 47656 [details]
Patch
Comment 2 Daniel Cheng 2010-02-04 15:36:46 PST
After talking with dimich, I have decided to refactor the Clipboard class instead.