Bug 15305 - Crash when dragging an image when "load images automatically" preference is off
Summary: Crash when dragging an image when "load images automatically" preference is off
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Images (show other bugs)
Version: 523.x (Safari 3)
Hardware: PC Windows XP
: P1 Normal
Assignee: Adam Roben (:aroben)
URL:
Keywords: InRadar, PlatformOnly
Depends on:
Blocks:
 
Reported: 2007-09-28 08:36 PDT by 808caaa4.8ce9.9cd6c799e9f6
Modified: 2007-10-07 17:23 PDT (History)
1 user (show)

See Also:


Attachments
Proposed fix with ChangeLog (2.71 KB, patch)
2007-10-07 17:08 PDT, Adam Roben (:aroben)
eric: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description 808caaa4.8ce9.9cd6c799e9f6 2007-09-28 08:36:43 PDT
Anonymous reporter(2ch.net, ID:tLTFSr9l0) said
Safari/WebKit seems crash when users begin drag at image-with-anchor objects.
With ntsd, it's inside WebCore::writeImageToDataObject().
Comment 1 Alexey Proskuryakov 2007-10-05 03:53:00 PDT
I do not think this is enough information to work on this bug. Is there an example URL where this crash can be seen?
Comment 2 808caaa4.8ce9.9cd6c799e9f6 2007-10-05 06:24:34 PDT
Thank you so much for pointing out, and sorry for inaccuracy.
Summary fixed slightly. Please rename properly if validated.

1. Open www.apple.com, new iPod nanos shown up today.
2. Drag over it, nothing wrong(OK).
3. disable Edit->Preferences->Appearance->"Display images when the page opens."
4. Reload www.apple.com, assume no pictures remains, includes iPod nanos.
5. Drag over the same area.
6. Crash may appear with my environment.

Comment 3 Alexey Proskuryakov 2007-10-05 07:08:53 PDT
Thank you for the additional information!

I could reproduce this with r24749 nightly (haven't downloaded anything newer on Windows).
Comment 4 David Kilzer (:ddkilzer) 2007-10-05 09:03:19 PDT
<rdar://problem/5524774>
Comment 5 Adam Roben (:aroben) 2007-10-07 17:08:02 PDT
Created attachment 16581 [details]
Proposed fix with ChangeLog
Comment 6 Eric Seidel (no email) 2007-10-07 17:12:36 PDT
Comment on attachment 16581 [details]
Proposed fix with ChangeLog

Looks good.
Comment 7 Adam Roben (:aroben) 2007-10-07 17:23:26 PDT
Landed as r26102