Bug 21402 - Add a clip in RenderView::layout to avoid doing offscreen paint
Summary: Add a clip in RenderView::layout to avoid doing offscreen paint
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-10-06 08:39 PDT by Julien Chaffraix
Modified: 2010-06-10 16:04 PDT (History)
1 user (show)

See Also:


Attachments
Proposed fix: add a clip to the viewRect (1.43 KB, patch)
2008-10-06 08:44 PDT, Julien Chaffraix
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Julien Chaffraix 2008-10-06 08:39:59 PDT
Currently in RenderView.cpp, there is a FIXME about adding a clip to the LayoutState.

Path forthcoming.
Comment 1 Julien Chaffraix 2008-10-06 08:44:15 PDT
Created attachment 24114 [details]
Proposed fix: add a clip to the viewRect
Comment 2 Eric Seidel (no email) 2008-10-06 14:45:16 PDT
Comment on attachment 24114 [details]
Proposed fix: add a clip to the viewRect

This one is for Hyatt.
Comment 3 Julien Chaffraix 2008-10-07 02:55:23 PDT
Comment on attachment 24114 [details]
Proposed fix: add a clip to the viewRect

As discussed with David Hyatt and Dan Bernstein, the win is odd and should be confirmed on ToT (I had done some rough testing on ToT and good experiment on an old bramch) before thinking of integrating this patch.

Clearing the review flag to give me some time for performance experiments on ToT.