NEW 211222
Web views should use black fill color before first paint on watchOS 
https://bugs.webkit.org/show_bug.cgi?id=211222
Summary Web views should use black fill color before first paint on watchOS
James Savage
Reported 2020-04-29 22:14:25 PDT
This better matches the platform and avoid flickering. <rdar://problem/58652428>
Attachments
Patch (1.72 KB, patch)
2020-04-30 18:36 PDT, James Savage 		no flags
DetailsFormatted DiffDiff
Patch (3.18 KB, patch)
2020-04-30 18:40 PDT, James Savage 		no flags
DetailsFormatted DiffDiff
Patch (11.80 KB, patch)
2020-05-13 20:48 PDT, James Savage 		james.savage: review?
james.savage: commit-queue?
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
James Savage
Comment 1 2020-04-30 18:36:46 PDT
Created attachment 398132 [details] Patch
James Savage
Comment 2 2020-04-30 18:40:33 PDT
Created attachment 398135 [details] Patch
Simon Fraser (smfr)
Comment 3 2020-04-30 20:38:47 PDT
Comment on attachment 398135 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=398135&action=review > Source/WebCore/page/FrameView.cpp:2990 > +#elif PLATFORM(WATCHOS) Can we do this without a platform #ifdef in this cross-platform file? > Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm:540 > + [self setBackgroundColor:[UIColor blackColor]]; What if the client has set _drawsBackground to NO?
Simon Fraser (smfr)
Comment 4 2020-04-30 20:41:43 PDT
Comment on attachment 398135 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=398135&action=review > Source/WebKit/ChangeLog:11 > + the background color for web views. This will then get propogated to propogated
James Savage
Comment 5 2020-05-13 20:48:48 PDT
Created attachment 399329 [details] Patch
Sam Weinig
Comment 6 2020-05-14 09:24:55 PDT
Comment on attachment 399329 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=399329&action=review > Source/WebCore/style/UserAgentStyle.cpp:101 > +#define SYSTEM_BACKGROUND_COLOR "html{background-color: white;}" no need for the space or ;
Note You need to log in before you can comment on or make changes to this bug.