New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 597965 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 599573
Owner:
Closed: Mar 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Fast-path and slow-path of paint invalidation rect mapping maps differently inside vertical-rl

Project Member Reported by wangxianzhu@chromium.org, Mar 25 2016

Issue description

fast/box-shadow/shadow-box-resize-writing-mode.html

<div style="-webkit-writing-mode: vertical-rl"></div>

If forced to go through slow-path paint invalidation rect mapping, the div will invalidate its own rect incorrectly flipped.

Now we flip for writing mode in mapToVisibleRectForPaintInvalidation() for own rect. I think this is similar to scroll offset and clipping that should be applied on child rects only, so we should do the flipping in child's mapToVisibleRectForPaintInvalidation().
 
Summary: Fast-path and slow-path of paint invalidation rect mapping maps differently inside vertical-rl (was: Fast-path and slow-path of paint invalidation rect mapping maps inside vertical-rl)
Owner: wangxianzhu@chromium.org
Status: Started (was: Available)
Status: WontFix (was: Started)
Invalid bug. We should not use fast-path for flipped blocks writing mode. Slow-path is correct.
Status: Assigned (was: WontFix)
This seems still a bug. Adding some unit tests.
Mergedinto: 599573
Status: Duplicate (was: Assigned)
This is actually caused by wrong box-shadow visual overflow (missing flipping), not "extra flipping" in mapToVisualRectInAncestorSpace().

Sign in to add a comment