Bug 81676 - [chromium] Use opaque contents for paint culling
Summary: [chromium] Use opaque contents for paint culling
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: Dana Jansens
URL:
Keywords:
Depends on:
Blocks: 81689
  Show dependency treegraph
 
Reported: 2012-03-20 10:13 PDT by Dana Jansens
Modified: 2012-03-20 12:51 PDT (History)
7 users (show)

See Also:


Attachments
Patch (11.76 KB, patch)
2012-03-20 10:18 PDT, Dana Jansens
no flags Details | Formatted Diff | Diff
Patch (5.80 KB, patch)
2012-03-20 11:52 PDT, Dana Jansens
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dana Jansens 2012-03-20 10:13:13 PDT
[chromium] Use opaque contents for paint culling
Comment 1 Dana Jansens 2012-03-20 10:16:46 PDT
Removes FIXME's that blocked this functionality, and adds a unit test to verify it's on.
Comment 2 Dana Jansens 2012-03-20 10:18:40 PDT
Created attachment 132846 [details]
Patch
Comment 3 Adrienne Walker 2012-03-20 11:19:27 PDT
Comment on attachment 132846 [details]
Patch

Forgive me for being conservative, but can you leave the setUsePaintTracking() code for now and we can remove it post m19 branch?
Comment 4 Dana Jansens 2012-03-20 11:52:13 PDT
Created attachment 132865 [details]
Patch

Sure :)
Comment 5 Adrienne Walker 2012-03-20 11:58:25 PDT
Comment on attachment 132865 [details]
Patch

Thanks!
Comment 6 WebKit Review Bot 2012-03-20 12:51:36 PDT
Comment on attachment 132865 [details]
Patch

Clearing flags on attachment: 132865

Committed r111426: <http://trac.webkit.org/changeset/111426>
Comment 7 WebKit Review Bot 2012-03-20 12:51:41 PDT
All reviewed patches have been landed.  Closing bug.