Bug 77496

Summary: Move the repainting logic out of RenderLayer
Product: WebKit Reporter: Julien Chaffraix <jchaffraix>
Component: Layout and RenderingAssignee: Julien Chaffraix <jchaffraix>
Status: NEW    
Severity: Normal CC: gustavo, hyatt, jamesr, pnormand, shezbaig.wk, simon.fraser, xan.lopez
Priority: P3    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Split the class, waiting for feedback on the name before adding all our build systems. jchaffraix: review-, pnormand: commit-queue-

Julien Chaffraix
Reported 2012-01-31 17:14:50 PST
Currently RenderLayer is a hub of different features that could be split into different classes with more dedicated purposes. Repainting and repaint rectangles caching are related functionalities that could live in their own class.
Attachments
Split the class, waiting for feedback on the name before adding all our build systems. (22.61 KB, patch)
2012-01-31 17:25 PST, Julien Chaffraix 		jchaffraix: review-
pnormand: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Julien Chaffraix
Comment 1 2012-01-31 17:25:41 PST
Created attachment 124858 [details] Split the class, waiting for feedback on the name before adding all our build systems.
Philippe Normand
Comment 2 2012-01-31 18:13:57 PST
Comment on attachment 124858 [details] Split the class, waiting for feedback on the name before adding all our build systems. Attachment 124858 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/11389233
Early Warning System Bot
Comment 3 2012-01-31 18:40:01 PST
Comment on attachment 124858 [details] Split the class, waiting for feedback on the name before adding all our build systems. Attachment 124858 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/11389251
Gyuyoung Kim
Comment 4 2012-01-31 23:01:40 PST
Comment on attachment 124858 [details] Split the class, waiting for feedback on the name before adding all our build systems. Attachment 124858 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/11389336
Julien Chaffraix
Comment 5 2012-06-19 10:25:53 PDT
Comment on attachment 124858 [details] Split the class, waiting for feedback on the name before adding all our build systems. Clearing review flags as it would need to be rebaselined. I don't have any bandwidth to do that at the moment.
Note You need to log in before you can comment on or make changes to this bug.