We need a patch to turn this on.
<rdar://problem/45885977>
Created attachment 354156 [details] Patch
Comment on attachment 354156 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=354156&action=review > Source/WebKit/UIProcess/ResourceLoadStatisticsMemoryStore.cpp:839 > RunLoop::main().dispatch([store = makeRef(m_store), seconds] () { updateCacheMaxAgeCap(); ?
(In reply to Chris Dumez from comment #3) > Comment on attachment 354156 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=354156&action=review > > > Source/WebKit/UIProcess/ResourceLoadStatisticsMemoryStore.cpp:839 > > RunLoop::main().dispatch([store = makeRef(m_store), seconds] () { > > updateCacheMaxAgeCap(); ? Yes, that's a nice enhancement. I tried first to chain them the other way around but decided I didn't want to set the member variable on every call to updateCacheMaxAgeCap(). Thanks for the review!
I will wait for the bots to run the tests before landing.
The Mac ImageOnlyFailure test issue is unrelated.
Created attachment 354170 [details] Patch for landing
Comment on attachment 354170 [details] Patch for landing Clearing flags on attachment: 354170 Committed r237949: <https://trac.webkit.org/changeset/237949>
All reviewed patches have been landed. Closing bug.