Stop using lambdas for WebResourceLoadStatisticsStore to interact with its WebsiteDataStore. Instead, WebResourceLoadStatisticsStore now holds a weak pointer to its WebsiteDataStore and is able to call methods on it directly. Reducing the indirection makes the code less complex and more understandable.
Created attachment 343880 [details] Patch
Created attachment 343881 [details] Patch
Created attachment 343882 [details] Patch
Comment on attachment 343882 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=343882&action=review r=me > Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.cpp:1480 > + m_resourceLoadStatistics->setStatisticsTestingCallback(WTFMove(callback)); Yes, I like this much better. We only worry about the testing callback in the one case where we need it.
Comment on attachment 343882 [details] Patch r=me
Comment on attachment 343882 [details] Patch Clearing flags on attachment: 343882 Committed r233359: <https://trac.webkit.org/changeset/233359>
All reviewed patches have been landed. Closing bug.
<rdar://problem/41639643>