RESOLVED FIXED 165186
REGRESSION (r209104): storage/indexeddb/modern/gc-closes-database(-private).html timeout 
https://bugs.webkit.org/show_bug.cgi?id=165186
Summary REGRESSION (r209104): storage/indexeddb/modern/gc-closes-database(-private).h...
Brady Eidson
Reported 2016-11-29 19:48:01 PST
REGRESSION (r209104): storage/indexeddb/modern/gc-closes-database(-private).html timeout I know why. 209104 was meant to resolve a GuardMalloc issue, which it did, but it also introduced a leak. I'll resolve both the leak and GuardMalloc issue at the same time in this bug.
Attachments
Add attachment proposed patch, testcase, etc.
Brady Eidson
Comment 1 2016-11-29 19:51:14 PST
Skipped the tests for now in https://trac.webkit.org/changeset/209114
Brady Eidson
Comment 2 2016-11-29 20:16:06 PST
I figured this out.
Brady Eidson
Comment 3 2016-11-29 20:29:43 PST
(In reply to comment #2) > I figured this out. Well, no... no I did not, I guess. Huh.
Chris Dumez
Comment 4 2016-11-29 20:37:36 PST
Reverted r209096, r209104, and r209114 for reason: Caused over 200 layout test failures on the bots Committed r209117: <http://trac.webkit.org/changeset/209117>
Brady Eidson
Comment 5 2016-11-29 23:21:46 PST
(In reply to comment #4) > Reverted r209096, r209104, and r209114 for reason: > > Caused over 200 layout test failures on the bots > > Committed r209117: <http://trac.webkit.org/changeset/209117> Have a link to that...? (All EWS passed and I can't reproduce such failures locally...?)
Brady Eidson
Comment 6 2016-11-30 11:10:30 PST
With the rollouts last night and then https://trac.webkit.org/changeset/209132 this morning, this has been resolved.
Note You need to log in before you can comment on or make changes to this bug.