Summary: | [Chromium] fast/loader/stateobjects/state-api-on-detached-frame-crash.html does not pass on Chromium | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mihai Parparita <mihaip> | ||||||
Component: | Tools / Tests | Assignee: | Mihai Parparita <mihaip> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, dglazkov, ojan, tony | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Mihai Parparita
2010-10-29 10:45:04 PDT
Created attachment 72348 [details]
Patch
Any of you mind reviewing? Comment on attachment 72348 [details]
Patch
It would be nice to add some text to the test describing what is being tested, but either way, this change seems fine.
(In reply to comment #3) > (From update of attachment 72348 [details]) > It would be nice to add some text to the test describing what is being tested, but either way, this change seems fine. True. It used to have a description before http://trac.webkit.org/changeset/53672/trunk/LayoutTests/fast/loader/stateobjects/state-api-on-detached-frame-crash.html, I'll add it back. Created attachment 72350 [details]
Patch
Comment on attachment 72350 [details] Patch Clearing flags on attachment: 72350 Committed r70913: <http://trac.webkit.org/changeset/70913> All reviewed patches have been landed. Closing bug. |