New issue
Advanced search Search tips

Issue 922940 link

Starred by 1 user

Issue metadata

Status: Duplicate
Owner: ----
Closed: Jan 17
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

Border radius is affecting the rendering of box-shadow on macOs

Reported by nathanmj...@gmail.com, Jan 17 (5 days ago)

Issue description

Chrome Version       : 71.0.3578.98 (Official Build) (64-bit)
URLs (if applicable) : https://jsfiddle.net/g3bfhr6x/30/
OS version               : 10.14.2 (18C54) Mojave
Behavior in Safari (if applicable): OK
Behavior in Firefox (if applicable): OK

What steps will reproduce the problem?
1. Open JS Fiddle provided
2. Observe the difference in the box-shadow for the white boxes (use screenshot as reference)

What is the expected result?
Border radius should not affect the rendering of box-shadow

What happens instead?
Border is affecting the rendering of box-shadow - see jsfiddle and screenshots for an example.

 
Screenshot 2019-01-17 at 12.27.34.png
4.6 KB View Download
Screenshot 2019-01-17 at 12.26.48.png
6.9 KB View Download

Comment 1 by dtapu...@chromium.org, Jan 17 (5 days ago)

Components: Blink>Paint

Comment 2 by schenney@chromium.org, Jan 17 (5 days ago)

Labels: -Pri-3 Pri-2
Status: Available (was: Unconfirmed)
Works fine in Canary M-73, broken in M-71.

Comment 3 by schenney@chromium.org, Jan 17 (5 days ago)

Cc: robertphillips@chromium.org fmalita@chromium.org
Components: -Blink>Paint Internals>GPU>Rasterization Internals>Skia
Status: Untriaged (was: Available)
This was somehow fixed by https://chromium.googlesource.com/chromium/src/+/909793b85dd2eb0bd37954ec50fbc86d547be50a

Possibly because it changed the shadow to avoid some bad situation.

The most likely regression patch was this one: https://chromium.googlesource.com/chromium/src/+/a2014453d14ec35d485140438224d26d6c1c9d94

Over to the hardware raster team to determine if the problem is still present at different radii sizes.

Comment 4 by fmalita@chromium.org, Jan 17 (5 days ago)

Mergedinto: 912595
Status: Duplicate (was: Untriaged)
Should be fixed now (https://skia-review.googlesource.com/c/skia/+/181902).

Sign in to add a comment