Summary: | [Windows] Scroll offset being applied to plugins during print operation | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brent Fulgham <bfulgham> | ||||||||
Component: | WebKit Misc. | Assignee: | Brent Fulgham <bfulgham> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | andersca, aroben, eric, hamaji, jhoneycutt, tkent | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | PC | ||||||||||
OS: | Windows XP | ||||||||||
Attachments: |
|
Description
Brent Fulgham
2010-05-28 11:23:09 PDT
The Windows implementation of PluginView::paintWindowedPluginIntoContext is only used during print and print preview operations for windowed plugins. The drawing location passed to the plugin is computed based on the current scroll position, which is wrong for printing operations as the page position is always the same. Created attachment 57353 [details]
Patch
Created attachment 57358 [details]
Patch
Don't we have ways to test printing changes like this these days? I guess this patch changes pixel output but doesn't change render tree, right? If so, we cannot test this patch without Bug 20011. We should get Anders and/or Jon to look at this. Comment on attachment 57358 [details]
Patch
We have printing tests these days - Brent can you create a testcase?
Created attachment 97676 [details]
Patch
Wow! This one got lost in the shuffle. Landed under https://trac.webkit.org/changeset/146941. |