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

Issue 650843 link

Starred by 1 user
Status: Fixed
Owner:
wangxianzhu@chromium.org
Closed: Oct 2016
Cc:
chrishtr@chromium.org
█ wkorman@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Blocking:
issue 646176



Sign in to add a comment

PaintPropertyTreeBuilder produces paintOffset for children of a vertical-rl container with scrollbars

Project Member Reported by wangxianzhu@chromium.org, Sep 27 2016 
<body style="margin: 0">
<div style="width: 100px; height: 100px; overflow: scroll; writing-mode: vertical-rl">
  <div id="a" style="width: 80px; background-color: blue">A</div>
</div>
</body>

a.topLeftLocation() is (20, 0) instead of (5, 0). The width of the vertial scrollbar is not considered.

This causes wrong result of a.offsetLeft.

Also causes wrong result of paint/invalidation/vertical-overflow-parent.html and paint/invalidation/vertical-overflow-same.html in slimmingPaintInvalidation mode.
rlscroll.html
340 bytes View Download

Comment 1 by wangxianzhu@chromium.org, Sep 27 2016

Summary: PaintPropertyTreeBuilder produces paintOffset for children of a vertical-rl container with scrollbars (was: LayoutBox::topLeftLocation() returns wrong result for children of a vertical-rl container with scrollbars)
I was wrong in #0. a.topLeftLocation() is (20, 0) because it is scrolled to the physical left by (15, 0) because of the scrollbar on the right. The failures of paint/invalidation/vertical-overflow-parent.html and paint/invalidation/vertical-overflow-same.html should be caused by PaintPropertyTreeBuilder.

Comment 2 by wangxianzhu@chromium.org, Oct 5 2016

Status: Fixed (was: Assigned)
Fixed by https://codereview.chromium.org/2371353002/.
►

Sign in to add a comment