WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 108048
storage/indexeddb tests are crashing
https://bugs.webkit.org/show_bug.cgi?id=108048
Summary
storage/indexeddb tests are crashing
Keishi Hattori
Reported
2013-01-27 21:20:57 PST
I rolled out
r140602
but it still keeps crashing and I can't figure it out. I am going to mark them as crashing.
Attachments
Add attachment
proposed patch, testcase, etc.
Keishi Hattori
Comment 1
2013-01-27 21:22:44 PST
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=storage%2Findexeddb%2Fopen-twice-workers.html%2Cstorage%2Findexeddb%2Ftransaction-complete-workers.html%2Cstorage%2Findexeddb%2Fpending-activity-workers.html%2Cstorage%2Findexeddb%2Fcursor-advance-workers.html%2Cstorage%2Findexeddb%2Findex-basics-workers.html%2Cstorage%2Findexeddb%2Ffactory-basics-workers.html%2Cstorage%2Findexeddb%2Fobjectstore-basics-workers.html%2Cstorage%2Findexeddb%2Fdeletedatabase-delayed-by-open-and-versionchange-workers.html
Joshua Bell
Comment 2
2013-01-28 12:03:46 PST
This is the suspect change in the range that introduced the crash on all the bots:
http://trac.webkit.org/changeset/140653
Which is a revert of:
http://trac.webkit.org/changeset/140783
But this dependent(?) Chromium change wasn't reverted...
http://src.chromium.org/viewvc/chrome?view=rev&revision=178734
Unfortunately, it's not reproducing for me at the moment. :P Maybe a WebKit/chromium/DEPS roll is involved? I'll keep plugging away at it.
Adam Barth
Comment 3
2013-01-28 12:07:59 PST
Ah! Sorry for causing a mess.
Joshua Bell
Comment 4
2013-01-28 12:09:13 PST
(In reply to
comment #3
)
> Ah! Sorry for causing a mess.
My comment was poorly phrased. I don't have proof that's the cause, just a hunch. "This is *a* suspect change..." Still can't repro locally, which is itself odd. Alas, alec (who could) is out today.
Joshua Bell
Comment 5
2013-01-28 12:12:08 PST
Also, looks like WebKit/chromium/DEPS is at 178434 which is prior to the Chromium rev. The test crashes are also only happening on the ToT bots, not the DEPS bots.
Joshua Bell
Comment 6
2013-01-28 12:22:45 PST
My environment must have been mucked up. Reproducing easily now. Reverting
http://src.chromium.org/viewvc/chrome?view=rev&revision=178734
makes the crashes go away reliably as suspected. I'll revert that over on the chromium side.
Joshua Bell
Comment 7
2013-01-28 14:21:59 PST
Revert is finally picked up by the WebKit bots and they're starting to cycle green:
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=storage%2Findexeddb%2Fopen-twice-workers.html%2Cstorage%2Findexeddb%2Ftransaction-complete-workers.html%2Cstorage%2Findexeddb%2Fpending-activity-workers.html%2Cstorage%2Findexeddb%2Fcursor-advance-workers.html%2Cstorage%2Findexeddb%2Findex-basics-workers.html%2Cstorage%2Findexeddb%2Ffactory-basics-workers.html%2Cstorage%2Findexeddb%2Fobjectstore-basics-workers.html%2Cstorage%2Findexeddb%2Fdeletedatabase-delayed-by-open-and-versionchange-workers.html
Calling this good. I'll revert the other reverts.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug