RESOLVED FIXED Bug 206035
Check the existence of the optional m_sessionID before using it in WebProcess::setResourceLoadStatisticsEnabled() 
https://bugs.webkit.org/show_bug.cgi?id=206035
Summary Check the existence of the optional m_sessionID before using it in WebProcess...
John Wilander
Reported 2020-01-09 14:12:06 PST
Since m_sessionID is optional, we need to check that it exists before asking it whether the session is ephemeral.
Attachments
Patch (1.71 KB, patch)
2020-01-09 14:14 PST, John Wilander 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
John Wilander
Comment 1 2020-01-09 14:12:20 PST
<rdar://problem/58455306>
John Wilander
Comment 2 2020-01-09 14:14:42 PST
Created attachment 387271 [details] Patch
Brent Fulgham
Comment 3 2020-01-09 15:02:51 PST
Comment on attachment 387271 [details] Patch r=me
WebKit Commit Bot
Comment 4 2020-01-09 15:47:42 PST
Comment on attachment 387271 [details] Patch Clearing flags on attachment: 387271 Committed r254310: <https://trac.webkit.org/changeset/254310>
WebKit Commit Bot
Comment 5 2020-01-09 15:47:43 PST
All reviewed patches have been landed. Closing bug.
Jonathan Bedard
Comment 6 2020-01-10 13:46:09 PST
I'm not 100% sure this is the cause, but we have a flakey regression in a resource load statistics test, and this seems like a plausible candidate: https://results.webkit.org/?suite=layout-tests&test=http%2Ftests%2FresourceLoadStatistics%2Fclassify-as-non-prevalent-based-on-sub-frame-under-top-frame-origins-database.html
Jonathan Bedard
Comment 7 2020-01-10 13:47:23 PST
Although <https://trac.webkit.org/changeset/254296> is also a plausible candidate.
John Wilander
Comment 8 2020-01-10 15:08:42 PST
Thanks for flagging, Jonathan! I'm trying to address the flakiness in https://bugs.webkit.org/show_bug.cgi?id=206100.
John Wilander
Comment 9 2020-01-14 10:49:19 PST
The flaky test looks all green after my latest change.
Jonathan Bedard
Comment 10 2020-01-14 11:22:49 PST
Looks good so far...still need a few more test runs to be confident.
Jonathan Bedard
Comment 11 2020-01-16 15:02:01 PST
I think we can pretty confidently say things are fixed now.
Note You need to log in before you can comment on or make changes to this bug.