RESOLVED FIXED 166970
[SOUP] Do not use the legacy session ID in WebFrameNetworkingContext::storageSession() 
https://bugs.webkit.org/show_bug.cgi?id=166970
Summary [SOUP] Do not use the legacy session ID in WebFrameNetworkingContext::storage...
Carlos Garcia Campos
Reported 2017-01-12 05:35:27 PST
We should get the actual session like we do in the network process.
Attachments
Patch (1.69 KB, patch)
2017-01-12 05:37 PST, Carlos Garcia Campos 		svillar: review+
buildbot: commit-queue-
DetailsFormatted DiffDiff
Archive of layout-test-results from ews104 for mac-elcapitan-wk2 (573.91 KB, application/zip)
2017-01-12 07:49 PST, Build Bot 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Carlos Garcia Campos
Comment 1 2017-01-12 05:37:20 PST
Created attachment 298673 [details] Patch
Sergio Villar Senin
Comment 2 2017-01-12 06:27:28 PST
Comment on attachment 298673 [details] Patch sgtm provided the mac failures are unrelated. BTW isn't this testable? Wasn't it causing any issues?
Carlos Garcia Campos
Comment 3 2017-01-12 06:34:27 PST
(In reply to comment #2) > Comment on attachment 298673 [details] > Patch > > sgtm provided the mac failures are unrelated. > > BTW isn't this testable? Wasn't it causing any issues? This is only used for the very few loads that still happen in the web process, and affects the private browsing mode that we don't test AFAIK.
Build Bot
Comment 4 2017-01-12 07:49:55 PST
Comment on attachment 298673 [details] Patch Attachment 298673 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/2877587 Number of test failures exceeded the failure limit.
Build Bot
Comment 5 2017-01-12 07:49:59 PST
Created attachment 298681 [details] Archive of layout-test-results from ews104 for mac-elcapitan-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews104 Port: mac-elcapitan-wk2 Platform: Mac OS X 10.11.6
Carlos Garcia Campos
Comment 6 2017-01-13 00:18:31 PST
Committed r210728: <http://trac.webkit.org/changeset/210728>
Note You need to log in before you can comment on or make changes to this bug.