Summary: | Background thread with ITP Database should lock when the network process is suspended | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kate Cheney <katherine_cheney> | ||||||||||
Component: | WebKit Misc. | Assignee: | Kate Cheney <katherine_cheney> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | beidson, bfulgham, cdumez, commit-queue, ggaren, katherine_cheney, sihui_liu, webkit-bug-importer, wilander | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | WebKit Nightly Build | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
Attachments: |
|
Description
Kate Cheney
2020-01-22 10:04:00 PST
The correct radar is <rdar://problem/58713379> Created attachment 388440 [details]
Patch
Created attachment 388496 [details]
Patch
Bots were green but this was actually causing flaky API test failures. This patch should fix them. I suggest we just queue a task to hang the background queue, like we do in StorageManagerSet::suspend(). I believe this would make things a lot simpler. Created attachment 388587 [details]
Patch
Comment on attachment 388587 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=388587&action=review > Source/WebKit/NetworkProcess/Classifier/WebResourceLoadStatisticsStore.cpp:1167 > + postTask([this, protectedThis = makeRef(*this), completionHandler = completionHandlerCaller.release()] () mutable { protectedThis is not needed, postTask() does this for you. (In reply to Chris Dumez from comment #8) > Comment on attachment 388587 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=388587&action=review > > > Source/WebKit/NetworkProcess/Classifier/WebResourceLoadStatisticsStore.cpp:1167 > > + postTask([this, protectedThis = makeRef(*this), completionHandler = completionHandlerCaller.release()] () mutable { > > protectedThis is not needed, postTask() does this for you. Got it, I'll remove before landing. Thanks! Created attachment 388589 [details]
Patch for landing
This is actually: <rdar://problem/58713379> Comment on attachment 388589 [details] Patch for landing Clearing flags on attachment: 388589 Committed r255039: <https://trac.webkit.org/changeset/255039> All reviewed patches have been landed. Closing bug. |