Summary: | (Crash) storage/indexeddb/intversion-open-in-upgradeneeded.html crashes intermittently on mac-wk2 | ||
---|---|---|---|
Product: | WebKit | Reporter: | Ryan Haddad <ryanhaddad> |
Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED DUPLICATE | ||
Severity: | Normal | CC: | beidson |
Priority: | P2 | Keywords: | InRadar |
Version: | WebKit Nightly Build | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=155050 | ||
Bug Depends on: | |||
Bug Blocks: | 154968 |
Description
Ryan Haddad
2016-03-07 13:43:40 PST
I think this was the "too many file handles open at once" problem. Which I fixed in http://trac.webkit.org/changeset/197474 (In reply to comment #3) > Which I fixed in http://trac.webkit.org/changeset/197474 Which was, sadly, a few days before Ryan filed this... =/ Grumble. Regardless, it is the same problem. With the entire test suite cruising along, and with the unpredictability of GC, and with some leaks still outstanding, we're clearly keeping some file handles open that we don't need. Fixing leaks is going to be one way out of this mess. Another is: https://bugs.webkit.org/show_bug.cgi?id=154923 According to the dashboard: - This test hasn't crashed in recent recorded history. - This test stopped timing out mid May-27th, which jives with my fix http://trac.webkit.org/changeset/201461 - This test is still flaking on text results quite often. My fix on 5/27 also jives with the nature of the crash log. I'm confident it made an impact here. The text diff is always about the blocked event: --- /Volumes/Data/slave/yosemite-debug-tests-wk2/build/layout-test-results/storage/indexeddb/intversion-open-in-upgradeneeded-expected.txt +++ /Volumes/Data/slave/yosemite-debug-tests-wk2/build/layout-test-results/storage/indexeddb/intversion-open-in-upgradeneeded-actual.txt @@ -34,8 +34,6 @@ onVersionChange(): db.close() -onBlocked(): - upgradeNeeded2(): db = event.target.result PASS event.newVersion is 3 ---- This is basically the same flakiness issue as https://bugs.webkit.org/show_bug.cgi?id=155050 This can be resolved with a test change (getting the blocked event is not relevant to the test, and blocked events are well tested elsewhere) Okay, hilarious - This is actually the SAME TEST as 155050. The change has already been made. We're good here. *** This bug has been marked as a duplicate of bug 155050 *** |