Summary: | Update NetworkStorageSession to support multiple persistent sessions and explicitly set cookie storages | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brady Eidson <beidson> | ||||||||||||
Component: | WebCore Misc. | Assignee: | Brady Eidson <beidson> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | achristensen, aestes, annulen, berto, buildbot, cgarcia, commit-queue, danw, gustavo, mcatanzaro, thorton, webkit-bug-importer | ||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||
Version: | WebKit Nightly Build | ||||||||||||||
Hardware: | Unspecified | ||||||||||||||
OS: | Unspecified | ||||||||||||||
Attachments: |
|
Description
Brady Eidson
2017-04-26 22:17:20 PDT
Created attachment 308337 [details]
Patch
Created attachment 308354 [details]
Patch
Created attachment 308357 [details]
Patch
Comment on attachment 308354 [details]
Patch
Oh boy
Created attachment 308395 [details]
Patch
(In reply to Alex Christensen from comment #4) > Comment on attachment 308354 [details] > Patch > > Oh boy It's on like Donkey Kong. Comment on attachment 308395 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=308395&action=review > Source/WebCore/platform/network/cf/NetworkStorageSessionCFNet.cpp:65 > + auto sharedCookieStorage = adoptCF(_CFHTTPCookieStorageGetDefault(kCFAllocatorDefault)); Over IRC we confirmed that what _CFHTTPCookieStorageGetDefault() returns shouldn't be adopted. > Source/WebCore/platform/network/cf/NetworkStorageSessionCFNet.cpp:72 > +NetworkStorageSession::NetworkStorageSession(SessionID sessionID, RetainPtr<CFURLStorageSessionRef> platformSession, RetainPtr<CFHTTPCookieStorageRef> platformCookieStorage) These RetainPtrs could probably be rvalue references. Created attachment 308407 [details]
Patch
Will CQ+ after I finish running API tests locally (damn you EWS!) Comment on attachment 308407 [details] Patch Clearing flags on attachment: 308407 Committed r215883: <http://trac.webkit.org/changeset/215883> All reviewed patches have been landed. Closing bug. Should SessionID::isEphemeral() be changed now? (In reply to Konstantin Tokarev from comment #12) > Should SessionID::isEphemeral() be changed now? That'll be done in https://bugs.webkit.org/show_bug.cgi?id=171367 |