RESOLVED FIXED 225979
REGRESSION(r277505): Crash under WTF::Detail::CallableWrapper<WebKit::WebPageProxy::setUnderPageBackgroundColorOverride(WebCore::Color&&)::$_10, void>::call 
https://bugs.webkit.org/show_bug.cgi?id=225979
Summary REGRESSION(r277505): Crash under WTF::Detail::CallableWrapper<WebKit::WebPage...
Devin Rousso
Reported 2021-05-19 13:03:22 PDT
.
Attachments
Patch (1.70 KB, patch)
2021-05-19 13:09 PDT, Devin Rousso 		thorton: review+
DetailsFormatted DiffDiff
Patch (1.71 KB, patch)
2021-05-20 14:18 PDT, Devin Rousso 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Devin Rousso
Comment 1 2021-05-19 13:03:38 PDT
<rdar://problem/78130098>
Devin Rousso
Comment 2 2021-05-19 13:09:19 PDT
Created attachment 429089 [details] Patch
Tim Horton
Comment 3 2021-05-20 14:12:59 PDT
Comment on attachment 429089 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=429089&action=review > Source/WebKit/UIProcess/WebPageProxy.cpp:1958 > + if (m_pageClient) > + didChangeBackgroundColor(); Hmm, I would expect it here if you had moved the dereferencing of m_pageClient here, but inside didChangeBackgroundColor if you didn't. Why this version?
Devin Rousso
Comment 4 2021-05-20 14:16:35 PDT
Comment on attachment 429089 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=429089&action=review >> Source/WebKit/UIProcess/WebPageProxy.cpp:1958 >> + didChangeBackgroundColor(); > > Hmm, I would expect it here if you had moved the dereferencing of m_pageClient here, but inside didChangeBackgroundColor if you didn't. Why this version? No particular reason. Just trying to keep the diff smaller I suppose. I'll add `m_pageClient->didChangeBackgroundColor()` here instead.
Devin Rousso
Comment 5 2021-05-20 14:18:49 PDT
Created attachment 429221 [details] Patch
EWS
Comment 6 2021-05-20 14:54:00 PDT
Committed r277827 (237973@main): <https://commits.webkit.org/237973@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 429221 [details].
Note You need to log in before you can comment on or make changes to this bug.