Bug 75717
| Summary: | [Chromium] Move setVisibleRect() calls for mask and replica layers into calculateDrawTransformAndVisibility() | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Xianzhu Wang <wangxianzhu> |
| Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | enne, shawnsingh, wjmaclean |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Xianzhu Wang
This is a followup of bug 72162 - Move setVisibleRect() calls into calculateDrawTransformAndVisibility().
setVisibleRect() calls for mask and replica layers are still in CCLayerTreeHost.cpp but they seem also belong to calculateDrawTransformAndVisibility().
Also some questions (copied from my comments to the patch of bug 72162 after it has been committed):
1. Why setVisibleLayerRect to the whole content bounds of the renderSurfaceLayer instead of the actual visible rect?
2. setVisibleLayerRect() for the replica layer seems lost after the patch.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Adrienne Walker
(In reply to comment #0)
> This is a followup of bug 72162 - Move setVisibleRect() calls into calculateDrawTransformAndVisibility().
> setVisibleRect() calls for mask and replica layers are still in CCLayerTreeHost.cpp but they seem also belong to calculateDrawTransformAndVisibility().
>
> Also some questions (copied from my comments to the patch of bug 72162 after it has been committed):
> 1. Why setVisibleLayerRect to the whole content bounds of the renderSurfaceLayer instead of the actual visible rect?
This sounds like an oversight. I can't think of why it would need to be this way.
> 2. setVisibleLayerRect() for the replica layer seems lost after the patch.
This sounds like it could be a bug.
Xianzhu Wang
The bug has been fixed in chromium.