Bug 75923

Summary: [chromium] Pasteboard::documentFragment should fall back to text if there's no HTML text
Product: WebKit Reporter: Daniel Cheng <dcheng>
Component: PlatformAssignee: Daniel Cheng <dcheng>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Daniel Cheng
Reported 2012-01-09 18:27:59 PST
This matches the behavior of the other ports and prevents buggy programs like gvim from breaking pasting in richly editable areas.
Attachments
Patch (1.89 KB, patch)
2012-01-09 18:48 PST, Daniel Cheng 		no flags
DetailsFormatted DiffDiff
Patch (4.45 KB, patch)
2012-01-10 13:55 PST, Daniel Cheng 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Daniel Cheng
Comment 1 2012-01-09 18:48:26 PST
Created attachment 121780 [details] Patch
Evan Martin
Comment 2 2012-01-09 20:42:33 PST
Comment on attachment 121780 [details] Patch You'll need a test before any reviewer will r+ this
Daniel Cheng
Comment 3 2012-01-09 21:04:09 PST
I'd need to add something to LayoutTestController (in the WebKit tree) which would poke at the internal bits of MockWebClipboardImpl (in the Chromium tree). If someone can suggest an elegant way to do this, then sure, but otherwise it's not feasible.
Daniel Cheng
Comment 4 2012-01-10 13:55:10 PST
Created attachment 121902 [details] Patch
Daniel Cheng
Comment 5 2012-01-10 13:56:02 PST
Comment on attachment 121902 [details] Patch This change requires a small change in the Chromium side, so CQ- until I land that change and roll the chromium deps in WebKit.
Daniel Cheng
Comment 6 2012-01-10 16:40:15 PST
I've landed the DEPS roll at http://trac.webkit.org/changeset/104650.
Daniel Cheng
Comment 7 2012-01-10 17:19:21 PST
Comment on attachment 121902 [details] Patch Clearing flags on attachment: 121902 Committed r104655: <http://trac.webkit.org/changeset/104655>
Daniel Cheng
Comment 8 2012-01-10 17:19:24 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.