outline-width property doesn't handle inherit value properly

RESOLVED DUPLICATE of bug 598080

Status

()

Core
CSS Parsing and Computation
RESOLVED DUPLICATE of bug 598080
6 months ago
6 months ago

People

(Reporter: canaltinova, Assigned: canaltinova)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

6 months ago
I found this while looking into web platform tests that stylo passes but gecko fails in bug 1389187. This causes `/css/CSS2/ui/outline-width-096.xht` failure on non-stylo build.

Currently gecko does a fixup to computed value of outline-width[1] and stores that value inside `mActualOutlineWidth` field instead of `mOutlineWidth` in `nsStyleOutline`. But inside nsRuleNode.cpp, we are still getting mOutlineWidth of parent[2] and that's causing to get the wrong inherited value.

[1] http://searchfox.org/mozilla-central/rev/e5b13e6224dbe3182050cf442608c4cb6a8c5c55/layout/style/nsStyleStruct.cpp#569-578
[2] http://searchfox.org/mozilla-central/rev/6482c8a5fa5c7446e82ef187d1a1faff49e3379e/layout/style/nsRuleNode.cpp#7943
Comment hidden (mozreview-request)
(Assignee)

Updated

6 months ago
Summary: outline-width property doesn't handle inherit value property → outline-width property doesn't handle inherit value properly
Comment on attachment 8897320 [details]
Bug 1390432 - Get the right parent outline-width value during computation

Patch looks trivial, but I don't know the cascade logic in the old style system, so deferring to dbaron.
Attachment #8897320 - Flags: review?(bobbyholley) → review?(dbaron)

Comment 3

6 months ago
mozreview-review
Comment on attachment 8897320 [details]
Bug 1390432 - Get the right parent outline-width value during computation

https://reviewboard.mozilla.org/r/168624/#review174124

So there are two differences between mOutlineWidth and mActualOutlineWidth (see nsStyleOutline::RecalcData):
 1. the fixup for outline-style:none
 2. the rounding to device pixels

(1) should be part of the computed value, but (2) should not.

bug 598080 already covers this problem.  See my review comments there, e.g., bug 598080 comment 14.
Attachment #8897320 - Flags: review?(dbaron) → review-
Status: NEW → RESOLVED
Last Resolved: 6 months ago
Resolution: --- → DUPLICATE
Duplicate of bug: 598080
You need to log in before you can comment on or make changes to this bug.