Bug 15624

Summary: Transforms don't work properly with opacity
Product: WebKit Reporter: Dave Hyatt <hyatt>
Component: Layout and RenderingAssignee: Dave Hyatt <hyatt>
Status: RESOLVED FIXED    
Severity: Normal CC: mitz
Priority: P2    
Version: 523.x (Safari 3)   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Test Case
none
Patch that makes transforms work with opacity.
hyatt: review-
Fix that aggressively begins ancestor transparency layers when a transform is encountered mitz: review+

Dave Hyatt
Reported 2007-10-22 12:03:42 PDT
The clip that is set prior to beginning a transparency layer is not transform-aware.
Attachments
Test Case (453 bytes, text/html)
2007-10-22 12:04 PDT, Dave Hyatt 		no flags
Details
Patch that makes transforms work with opacity. (5.68 KB, patch)
2007-10-22 12:06 PDT, Dave Hyatt 		hyatt: review-
DetailsFormatted DiffDiff
Fix that aggressively begins ancestor transparency layers when a transform is encountered (6.74 KB, patch)
2007-10-22 12:23 PDT, Dave Hyatt 		mitz: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Dave Hyatt
Comment 1 2007-10-22 12:04:15 PDT
Created attachment 16805 [details] Test Case
Dave Hyatt
Comment 2 2007-10-22 12:06:35 PDT
Created attachment 16806 [details] Patch that makes transforms work with opacity.
Dave Hyatt
Comment 3 2007-10-22 12:18:37 PDT
Comment on attachment 16806 [details] Patch that makes transforms work with opacity. This has a couple of minor issues. I'm going to polish it.
Dave Hyatt
Comment 4 2007-10-22 12:23:35 PDT
Created attachment 16807 [details] Fix that aggressively begins ancestor transparency layers when a transform is encountered
mitz
Comment 5 2007-10-22 14:33:11 PDT
Comment on attachment 16807 [details] Fix that aggressively begins ancestor transparency layers when a transform is encountered r=me provided you remove the unused argument to beginTransparencyLayers() and add a change log
mitz
Comment 6 2007-10-22 17:32:55 PDT
Landed in <http://trac.webkit.org/projects/webkit/changeset/26894>.
Note You need to log in before you can comment on or make changes to this bug.