Bug 68297 - Remove unused members from LayerChromium
Summary: Remove unused members from LayerChromium
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-16 17:54 PDT by Antoine Labour
Modified: 2011-09-22 19:41 PDT (History)
5 users (show)

See Also:

Attachments
Patch (5.21 KB, patch)
2011-09-16 17:57 PDT, Antoine Labour 		no flags Details | Formatted Diff | Diff
Patch (5.24 KB, patch)
2011-09-22 13:39 PDT, Antoine Labour 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Antoine Labour 2011-09-16 17:54:10 PDT 
Remove unused members from LayerChromium
Comment 1 Antoine Labour 2011-09-16 17:57:44 PDT 
Created attachment 107756 [details]
Patch
Comment 2 James Robinson 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.
Comment 3 Antoine Labour 2011-09-22 13:39:39 PDT 
Created attachment 108392 [details]
Patch
Comment 4 Antoine Labour 2011-09-22 13:40:23 PDT 
Rebased to ToT.
Comment 5 James Robinson 2011-09-22 13:49:18 PDT 
Comment on attachment 108392 [details]
Patch

R=me
Comment 6 WebKit Review Bot 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>
Comment 7 WebKit Review Bot 2011-09-22 19:41:06 PDT 
All reviewed patches have been landed.  Closing bug.