Bug 209644 – Deduplicate WebsiteDataStore::parameters() of Cocoa port and non-Cocoa port

• Component: WebKit2
• Status: RESOLVED FIXED

[Fujii Hironori]

[WinCairo] ASSERT(m_resourceLoadStatistics) is failing setThirdPartyCookieBlockingMode

WinCairo WTR is still failing the assertion even after r259076 which fixed the same assertion failure.

WinCairo, debug builds, trunk@259081

fast/dom/window-open-ephemeral.html
http/tests/adClickAttribution/store-disabled-in-ephemeral-session.html
http/tests/cache/disk-cache/disk-cache-vary-cookie-private.html
http/tests/security/cookies/third-party-cookie-blocking-redirect.html
http/tests/security/cookies/third-party-cookie-blocking-user-action.html
http/tests/security/cookies/third-party-cookie-blocking.html
storage/indexeddb/aborted-versionchange-closes-private.html
storage/indexeddb/basics-private.html
storage/indexeddb/clone-exception-private.html
storage/indexeddb/closed-cursor-private.html
storage/indexeddb/connection-leak-private.html


> ASSERTION FAILED: m_resourceLoadStatistics
> ..\..\Source\WebKit\NetworkProcess/NetworkSession.cpp(248) : setThirdPartyCookieBlockingMode
> 1   00007FF926951429 WTFCrash
> 2   00007FF919A21F50 WTFCrashWithInfo
> 3   00007FF91D8B74FD WebKit::NetworkSession::setThirdPartyCookieBlockingMode
> 4   00007FF91D819878 WebKit::NetworkProcess::setShouldBlockThirdPartyCookiesForTesting
> 5   00007FF91D4986A4 IPC::callMemberFunctionImpl<WebKit::NetworkProcess,void (WebKit::NetworkProcess::*)(PAL::SessionID, WebCore::ThirdPartyCookieBlockingMode, WTF::CompletionHandler<void ()> &&),void (),std::tuple<PAL::SessionID,WebCore::ThirdPartyCookieBlockingMode>,0,1>
> 6   00007FF91D496A14 IPC::callMemberFunction<WebKit::NetworkProcess,void (WebKit::NetworkProcess::*)(PAL::SessionID, WebCore::ThirdPartyCookieBlockingMode, WTF::CompletionHandler<void ()> &&),void (),std::tuple<PAL::SessionID,WebCore::ThirdPartyCookieBlockingMode>,std::integer_sequence<unsigned long long,0,1> >
> 7   00007FF91D432B33 IPC::handleMessageAsync<Messages::NetworkProcess::SetShouldBlockThirdPartyCookiesForTesting,WebKit::NetworkProcess,void (WebKit::NetworkProcess::*)(PAL::SessionID, WebCore::ThirdPartyCookieBlockingMode, WTF::CompletionHandler<void ()> &&)>
> 8   00007FF91D427021 WebKit::NetworkProcess::didReceiveNetworkProcessMessage
> 9   00007FF91D81278C WebKit::NetworkProcess::didReceiveMessage
> 10  00007FF91DA3B9F6 IPC::Connection::dispatchMessage
> 11  00007FF91DA3C0DB IPC::Connection::dispatchMessage
> 12  00007FF91DA3C78B IPC::Connection::dispatchOneIncomingMessage
> 13  00007FF91DA3F50C IPC::Connection::enqueueIncomingMessage::<unnamed-tag>::operator()
> 14  00007FF91DA3F467 WTF::Detail::CallableWrapper<`lambda at ..\..\Source\WebKit\Platform\IPC\Connection.cpp:981:30',void>::call
> 15  00007FF9269645D0 WTF::Function<void ()>::operator()
> 16  00007FF9269C0BBC WTF::RunLoop::performWork
> 17  00007FF926A8D4D5 WTF::RunLoop::wndProc
> 18  00007FF926A8D3E9 WTF::RunLoop::RunLoopWndProc
> 19  00007FF947735C0D CallWindowProcW
> 20  00007FF947735602 DispatchMessageW
> 21  00007FF926A8D6F3 WTF::RunLoop::run
> 22  00007FF91D2DFD8B WebKit::AuxiliaryProcessMain<WebKit::NetworkProcess,WebKit::NetworkProcessMainCurl>
> 23  00007FF91D2DFB1B WebKit::NetworkProcessMain
> 24  00007FF7D6CE101C main
> 25  00007FF7D6CE123C __scrt_common_main_seh
> 26  00007FF948FF7BD4 BaseThreadInitThunk
> 27  00007FF94914CED1 RtlUserThreadStart
> LEAK: 20 WebPageProxy

[Fujii Hironori]

Created attachment 394889 [details]
Patch

[youenn fablet]

Comment on attachment 394889 [details]
Patch

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

> Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.cpp:2221
> +    };

There is a lot of code shared with WebsiteDataStoreParameters WebsiteDataStore::parameters() in WebsiteDataStoreCocoa.mm.
Can we try sharing it?
GTP seems to have an issue with hasStatisticsTestingCallback.

[Fujii Hironori]

Comment on attachment 394889 [details]
Patch

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

>> Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.cpp:2221
>> +    };
> 
> There is a lot of code shared with WebsiteDataStoreParameters WebsiteDataStore::parameters() in WebsiteDataStoreCocoa.mm.
> Can we try sharing it?
> GTP seems to have an issue with hasStatisticsTestingCallback.

Good idea. Will do so.

[Fujii Hironori]

Created attachment 395005 [details]
Patch

[Fujii Hironori]

Created attachment 395009 [details]
Patch

[Fujii Hironori]

Created attachment 395012 [details]
Patch

[Fujii Hironori]

Created attachment 395013 [details]
Patch

[Fujii Hironori]

Created attachment 395017 [details]
Patch

[Fujii Hironori]

Comment on attachment 395017 [details]
Patch

Clearing flags on attachment: 395017

Committed r259323: <https://trac.webkit.org/changeset/259323>

[Fujii Hironori]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/61131748>