Summary: | Add ENABLE_CUSTOM_SCROLLBARS and define it for macOS and for non-Cocoa platforms | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> | ||||||||
Component: | New Bugs | Assignee: | Simon Fraser (smfr) <simon.fraser> | ||||||||
Status: | ASSIGNED --- | ||||||||||
Severity: | Normal | CC: | benjamin, cdumez, cmarcelo, darin, esprehn+autocc, ews-watchlist, fred.wang, glenn, kondapallykalyan, mifenton, mmaxfield, pdr, simon.fraser, thorton, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | 210479 | ||||||||||
Bug Blocks: | 210438 | ||||||||||
Attachments: |
|
Description
Simon Fraser (smfr)
2020-04-13 15:29:00 PDT
Created attachment 396343 [details]
Patch
This fix wasn't complete. Created attachment 396374 [details]
Patch
Comment on attachment 396374 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=396374&action=review > Source/WTF/wtf/PlatformEnable.h:840 > -#if !defined(ENABLE_CUSTOM_SCROLLBARS) > +#if !defined(ENABLE_CUSTOM_SCROLLBARS) && !PLATFORM(COCOA) > #define ENABLE_CUSTOM_SCROLLBARS 1 > #endif This should be fixed in PlatformEnableCocoa.h instead, like this: #if !defined(ENABLE_CUSTOM_SCROLLBARS) && !PLATFORM(MAC) #define ENABLE_CUSTOM_SCROLLBARS 0 #endif PlatformEnableCocoa.h doesn't ever have to set it to 1. Created attachment 396378 [details]
Patch
Rolling out the first patch via bug 210479. I don't have time to address the iOS WK2 failures at present, will postpone this. |