Bug 68297

Summary: Remove unused members from LayerChromium
Product: WebKit Reporter: Antoine Labour <piman>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: enne, jamesr, nduca, vangelis, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Antoine Labour
Reported 2011-09-16 17:54:10 PDT
Remove unused members from LayerChromium
Attachments
Patch (5.21 KB, patch)
2011-09-16 17:57 PDT, Antoine Labour 		no flags
DetailsFormatted DiffDiff
Patch (5.24 KB, patch)
2011-09-22 13:39 PDT, Antoine Labour 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Antoine Labour
Comment 1 2011-09-16 17:57:44 PDT
Created attachment 107756 [details] Patch
James Robinson
Comment 2 2011-09-19 12:15:20 PDT
Comment on attachment 107756 [details] Patch This patch doesn't seem to apply cleanly to ToT, but it looks good. FWIW, zPosition is commented to be for debugging so we might want to use it someday. I think it's fine to remove for now - we can add if we need to.
Antoine Labour
Comment 3 2011-09-22 13:39:39 PDT
Created attachment 108392 [details] Patch
Antoine Labour
Comment 4 2011-09-22 13:40:23 PDT
Rebased to ToT.
James Robinson
Comment 5 2011-09-22 13:49:18 PDT
Comment on attachment 108392 [details] Patch R=me
WebKit Review Bot
Comment 6 2011-09-22 19:41:01 PDT
Comment on attachment 108392 [details] Patch Clearing flags on attachment: 108392 Committed r95776: <http://trac.webkit.org/changeset/95776>
WebKit Review Bot
Comment 7 2011-09-22 19:41:06 PDT
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.