Bug 165186
| Summary: | REGRESSION (r209104): storage/indexeddb/modern/gc-closes-database(-private).html timeout | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Brady Eidson <beidson> |
| Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | cdumez |
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | |||
| Bug Blocks: | 160306 | ||
Brady Eidson
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
Skipped the tests for now in https://trac.webkit.org/changeset/209114
Brady Eidson
I figured this out.
Brady Eidson
(In reply to comment #2)
> I figured this out.
Well, no... no I did not, I guess. Huh.
Chris Dumez
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
(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
With the rollouts last night and then https://trac.webkit.org/changeset/209132 this morning, this has been resolved.