RESOLVED FIXED 189336
[WinCairo][MiniBrowser][WebKit] REGRESSION(r235398): ASSERTION FAILED: !m_client.didReceiveTitleForFrame 
https://bugs.webkit.org/show_bug.cgi?id=189336
Summary [WinCairo][MiniBrowser][WebKit] REGRESSION(r235398): ASSERTION FAILED: !m_cli...
Fujii Hironori
Reported Thursday, September 6, 2018 11:46:29 AM UTC
[WinCairo][MiniBrowser][WebKit] REGRESSION(r235398): ASSERTION FAILED: !m_client.didReceiveTitleForFrame WinCairo MiniBrowser hits RELEASE_ASSERT failure since r235398. > ASSERTION FAILED: !m_client.didReceiveTitleForFrame > c:\webkit\gb\source\webkit\uiprocess\api\c\wkpage.cpp(1050) : WKPageSetPageLoaderClient::LoaderClient::LoaderClient
Attachments
WIP patch (3.21 KB, patch)
2018-09-06 04:06 PDT, Fujii Hironori 		no flags
DetailsFormatted DiffDiff
Patch (4.68 KB, patch)
2018-09-06 22:14 PDT, Fujii Hironori 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Fujii Hironori
Comment 1 Thursday, September 6, 2018 12:06:59 PM UTC
Created attachment 349016 [details] WIP patch Umm, there is no WK2 C API to call back the title. Cocoa and glib WK2 API exist.
Alex Christensen
Comment 2 Thursday, September 6, 2018 5:49:44 PM UTC
Looks good if it works for you. It might be worth adding a PageLoadState::Observer eventually.
Fujii Hironori
Comment 3 Friday, September 7, 2018 6:14:33 AM UTC
Created attachment 349113 [details] Patch
WebKit Commit Bot
Comment 4 Saturday, September 8, 2018 1:37:25 AM UTC
Comment on attachment 349113 [details] Patch Clearing flags on attachment: 349113 Committed r235824: <https://trac.webkit.org/changeset/235824>
WebKit Commit Bot
Comment 5 Saturday, September 8, 2018 1:37:26 AM UTC
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 6 Saturday, September 8, 2018 1:38:20 AM UTC
<rdar://problem/44246092>
Fujii Hironori
Comment 7 Wednesday, July 17, 2019 4:52:58 AM UTC
(In reply to Alex Christensen from comment #2) > Looks good if it works for you. It might be worth adding a > PageLoadState::Observer eventually. Filed: Bug 199848 – [Win][WebKit] Add PageLoadState::Observer C API
Note You need to log in before you can comment on or make changes to this bug.