Bug 180460

Summary: [Mac] REGRESSION (r224527): Drawing a path with box-shadow takes double the blur-radius on Retina display
Product: WebKit Reporter: Said Abou-Hallawa <sabouhallawa>
Component: Layout and RenderingAssignee: Said Abou-Hallawa <sabouhallawa>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, dbates, simon.fraser, thorton, webkit-bug-importer, zalan
Priority: P1 Keywords: InRadar, PlatformOnly, Regression
Version: WebKit Nightly Build   
Hardware: Mac   
OS: macOS 10.13   
Attachments:
Description Flags
test case
none
screen shot
none
Patch
none
Patch none

Description Said Abou-Hallawa 2017-12-05 17:07:23 PST
Created attachment 328528 [details]
test case

Open the attached test case on High Sierra MacOS Retina display.

Result: the box-shadow of the rounded rectangle does not render correctly. It looks as if the blur-radius is doubled (See the attached picture which compares the correct display and the wrong display).
Comment 1 Said Abou-Hallawa 2017-12-05 17:07:41 PST
Created attachment 328529 [details]
screen shot
Comment 2 Said Abou-Hallawa 2017-12-05 17:18:08 PST
Created attachment 328535 [details]
Patch
Comment 3 Radar WebKit Bug Importer 2017-12-05 17:20:30 PST
<rdar://problem/35869866>
Comment 4 Daniel Bates 2017-12-05 23:37:57 PST
Please update the title in the ChangeLog before landing. I added a space character between “REGRESSION” and ‘(‘. Please also add the radar bug URL under the Buzilla bug URLin the ChangeLog.
Comment 5 Said Abou-Hallawa 2017-12-06 09:16:35 PST
Created attachment 328577 [details]
Patch
Comment 6 WebKit Commit Bot 2017-12-06 09:31:40 PST
Comment on attachment 328577 [details]
Patch

Clearing flags on attachment: 328577

Committed r225575: <https://trac.webkit.org/changeset/225575>
Comment 7 WebKit Commit Bot 2017-12-06 09:31:41 PST
All reviewed patches have been landed.  Closing bug.
Comment 8 Simon Fraser (smfr) 2017-12-06 10:08:20 PST
Comment on attachment 328577 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=328577&action=review

> Source/WebCore/ChangeLog:10
> +        Disable using CGContextDrawPathDirect() for macOS High Sierra or earlier.
> +        This API has a bug when drawing a path with a shadow on Retina display.

You should reference the underlying radar.
Comment 9 Said Abou-Hallawa 2017-12-06 11:57:11 PST
Comment on attachment 328577 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=328577&action=review

>> Source/WebCore/ChangeLog:10
>> +        This API has a bug when drawing a path with a shadow on Retina display.
> 
> You should reference the underlying radar.

Done. <https://trac.webkit.org/changeset/225586>