Bug 193581 - LayoutTests/fast/inline/{continuation-outlines, continuation-outlines-with-layers}.html should have purple outline, but has blue outline
Summary: LayoutTests/fast/inline/{continuation-outlines, continuation-outlines-with-la...
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-01-18 10:25 PST by Daniel Bates
Modified: 2019-01-18 14:04 PST (History)
3 users (show)

See Also:


Attachments
[Screenshot] WebKit r240151 (67.28 KB, image/png)
2019-01-18 10:26 PST, Daniel Bates
no flags Details
[Screenshot] Mac Chrome Canary 73.0.3674.0 (73.80 KB, image/png)
2019-01-18 10:27 PST, Daniel Bates
no flags Details
[Screenshot] Mac Firefox 64.0.2 (68.30 KB, image/png)
2019-01-18 10:27 PST, Daniel Bates
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Bates 2019-01-18 10:25:56 PST
Compare the rendering of the color of the outline in LayoutTests/fast/inline/{continuation-outlines, continuation-outlines-with-layers}.html in Safari, Chrome, and Firefox.

ToT WebKit (r240151) renders the outline in the system blue. But Chrome Canary 73.0.3674.0 and Firefox 64.0.2 render them in purple.
Comment 1 Daniel Bates 2019-01-18 10:26:41 PST
Created attachment 359504 [details]
[Screenshot] WebKit r240151

Screenshot of rendering of LayoutTests/fast/inline/continuation-outlines.html.
Comment 2 Daniel Bates 2019-01-18 10:27:03 PST
Created attachment 359505 [details]
[Screenshot] Mac Chrome Canary 73.0.3674.0

Screenshot of rendering of LayoutTests/fast/inline/continuation-outlines.html.
Comment 3 Daniel Bates 2019-01-18 10:27:27 PST
Created attachment 359506 [details]
[Screenshot] Mac Firefox 64.0.2

Screenshot of rendering of LayoutTests/fast/inline/continuation-outlines.html.
Comment 4 Simon Fraser (smfr) 2019-01-18 11:24:47 PST
https://drafts.csswg.org/css-ui-3/#outline-style says:

The auto value permits the user agent to render a custom outline style, typically a style which is either a user interface default for the platform, or perhaps a style that is richer than can be described in detail in CSS, e.g. a rounded edge outline with semi-translucent outer pixels that appears to glow. As such, this specification does not define how the outline-color is incorporated or used (if at all) when rendering auto style outlines. User agents may treat auto as solid.
Comment 5 Daniel Bates 2019-01-18 14:04:31 PST
(In reply to Simon Fraser (smfr) from comment #4)
> https://drafts.csswg.org/css-ui-3/#outline-style says:
> 
> The auto value permits the user agent to render a custom outline style,
> typically a style which is either a user interface default for the platform,
> or perhaps a style that is richer than can be described in detail in CSS,
> e.g. a rounded edge outline with semi-translucent outer pixels that appears
> to glow. As such, this specification does not define how the outline-color
> is incorporated or used (if at all) when rendering auto style outlines. User
> agents may treat auto as solid.

I see. Then this behavior is intended though we should factor out this decision into a setting to untangle it from the Mac-specific animated focus ring drawing code. This will allow iOS to opt-into using the platform focus ring color over the CSS outline-color, but opt-out of using the animated drawFocusRing() overloads. Will add this setting as part of bug #193591.