New issue
Advanced search Search tips

Issue 637055 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Incorrect painting of body border for huge outline-offset values.

Project Member Reported by wkorman@chromium.org, Aug 11 2016

Issue description

While attempting to reland http://crrev.com/2225563002, sample failure:

https://storage.googleapis.com/chromium-layout-test-archives/linux_precise_blink_rel/716/layout-test-results/results.html

Red outline rect is incorrectly clipped to one tile.
 
http://crrev.com/2231623002 - Prevent integer overflow in ComputedStyle::outlineOutsetExtent.
Blocking: -529938
Cc: chrishtr@chromium.org
Labels: -Pri-2 Pri-3
Summary: Incorrect painting of body border for huge outline-offset values. (was: fast/css/outline-offset-large.html broken with rtree patch)
Don't believe this should block 529938. Expected behavior is not clear reviewing:

https://www.w3.org/TR/css-ui-3/#outline-offset

FireFox renders a straight red line at top of screen. Recent Chrome dev build 54.0.2816.0 dev without rtree patch renders with part of what looks like a thick black rect visible top-left and then red border surrounding the body content region.

RTree patch changes behavior in an arguably worse manner (all we see is one tile's worth of red border that would surround the body), but the case is quite obtuse.

Moving to P3 as an edge case where paint behavior is non-optimal and warrants review for correctness/consistency.

Owner: ----
Status: Available (was: Assigned)
Unassigning self from bugs that I don't expect to be able to get to soon in case someone else is able to pick them up.
Project Member

Comment 4 by sheriffbot@chromium.org, May 14 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Owner: schenney@chromium.org
Status: Assigned (was: Untriaged)
I think this works right now. I should re-enable and find out.

fast/css/outline-offset-large.html currently skipped.

Sign in to add a comment