| Summary: | [WinCairo] Rendering of themed elements on a layer with opacity produces nothing | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Martin Robinson <mrobinson> | ||||||||||
| Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> | ||||||||||
| Status: | RESOLVED FIXED | ||||||||||||
| Severity: | Normal | CC: | aroben, bfulgham | ||||||||||
| Priority: | P2 | ||||||||||||
| Version: | 528+ (Nightly build) | ||||||||||||
| Hardware: | PC | ||||||||||||
| OS: | Windows XP | ||||||||||||
| Bug Depends on: | |||||||||||||
| Bug Blocks: | 41113 | ||||||||||||
| Attachments: |
|
||||||||||||
Created attachment 59577 [details]
Rendering of test case
There appears to be two problems in GraphicsContext::releaseWindowsContext: 1. A cairo_save()/cairo_restore() pair is not used around the part of the method modifying the transformation matrix of the GraphicsContext's cairo surface. 2. The destination coordinates of the paint are affected by the scaling in the transformation matrix. I'm not sure if this is Cairo's expected behavior, but setting the destination coordinates during the call to cairo_set_source_surface() fixes the problem. Created attachment 59579 [details]
Patch
This patch looks good to me. Comment on attachment 59579 [details] Patch > + Functionality unchanged, no new tests. How can this be true? You're fixing a painting bug; that sounds like changed functionality to me. > + * platform/graphics/win/GraphicsContextCairoWin.cpp: > + (WebCore::GraphicsContext::releaseWindowsContext): > + Preform a cairo_save() and a cairo_restore() before modifying the transformation > + matrix of the Cairo surface. I think you mean "around", not "before". It's great to have detailed ChangeLog comments like these! The code change looks fine, but I'm still confused about the "Functionality unchanged" part. Created attachment 59815 [details]
Patch including manual test and ChangeLog fix
Comment on attachment 59815 [details] Patch including manual test and ChangeLog fix > +<p><b>BUG ID:</b> <a href="https://bugs.webkit.org/show_bug.cgi?id=41113">Bugzilla bug 41113</a> [WinCairo] Text box backgrounds do not render in partially opaque layers</p> This line is incorrect. If this version of the patch is otherwise satisfactory to the reviewer, I will change it to: > +<p><b>BUG ID:</b> <a href="https://bugs.webkit.org/show_bug.cgi?id=41111">Bugzilla bug 41113</a> [WinCairo] Rendering of themed elements on a layer with opacity produces nothing</p> Comment on attachment 59815 [details]
Patch including manual test and ChangeLog fix
r=me
Comment on attachment 59815 [details] Patch including manual test and ChangeLog fix Clearing flags on attachment: 59815 Committed r62011: <http://trac.webkit.org/changeset/62011> All reviewed patches have been landed. Closing bug. |
Created attachment 59576 [details] Failing test case Not only does the form element not render, even text which follows the form element fails to render. I've attached a test case that shows the issue I'm seeing.