Summary: | Crash on replacing document contents during drop | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Scott Violet <sky> | ||||||
Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | ||||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | PC | ||||||||
OS: | Windows XP | ||||||||
Attachments: |
|
Description
Scott Violet
2009-03-03 09:48:15 PST
Created attachment 28226 [details]
Fix for 24325
Comment on attachment 28226 [details]
Fix for 24325
This looks fine. I'm surprised that the test case needs to use waitUntilDone()
Ideally the test case should also set the text to PASSED when it's done. The blank page would confuse me at first.
Created attachment 28234 [details]
Fix for 24325
Changes output to be PASSED and removes waitForDone/NotifyDone.
Comment on attachment 28234 [details]
Fix for 24325
Looks fine.
Landed as http://trac.webkit.org/changeset/41403. |