Summary: | Crash under WebCore::TimerBase::~TimerBase after a download is canceled | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | David Quesada <david_quesada> | ||||
Component: | WebKit2 | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | achristensen, beidson, bfulgham, cdumez, commit-queue, jonlee, rniwa, sroberts, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=198298 | ||||||
Attachments: |
|
Description
David Quesada
2019-05-15 14:37:28 PDT
Created attachment 370001 [details]
Patch
Comment on attachment 370001 [details]
Patch
Looks sane to me.
Comment on attachment 370001 [details] Patch Clearing flags on attachment: 370001 Committed r245756: <https://trac.webkit.org/changeset/245756> All reviewed patches have been landed. Closing bug. Hello, it appears like after changes in r245756 the following API tests are flakey timeouts TestWebKitAPI.DownloadProgress.LoseProgressWhenDownloadIsCanceled Appears to time out every WK2 run https://build.webkit.org/builders/Apple%20Mojave%20Release%20WK1%20%28Tests%29/builds/5157/steps/run-api-tests/logs/stdio Also these 2 can also timeout on WK1 TestWebKitAPI.DownloadProgress.CancelDownloadWhenProgressIsCanceled TestWebKitAPI._WKDownload.DownloadMonitorCancel https://build.webkit.org/builders/Apple%20Mojave%20Release%20WK1%20%28Tests%29/builds/5158/steps/run-api-tests/logs/stdio We are having issues with Debug builders right now, so I do not have spades to test with. I will try to get some to test with, but this seems like the most likely culprit for these Timeouts. |