Bug 36247

Summary:

Dragging a PDF image triggers assertion in DragController::startDrag()

Product:

WebKit

Reporter:

Jian Li <jianli>

Component:

WebCore JavaScript

Assignee:

Jian Li <jianli>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

dglazkov

Priority:

Version:

528+ (Nightly build)

Hardware:

Mac

OS:

OS X 10.5

Attachments:

Description	Flags
Proposed Patch	dimich: review+, jianli: commit-queue-

Jian Li

Reported 2010-03-17 15:58:37 PDT

DragController.cpp:675: ASSERT(!image->filenameExtension().isEmpty()); We need to do the similar thing as in the fix for bug 32511: make PDFDocumentImage::filenameExtension() return a valid string.

Attachments
Proposed Patch (3.58 KB, patch) 2010-03-17 16:00 PDT, Jian Li	dimich: review+ jianli: commit-queue-	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Jian Li

Comment 1 2010-03-17 16:00:33 PDT

Created attachment 50967 [details] Proposed Patch

Dmitry Titov

Comment 2 2010-03-18 17:54:38 PDT

Comment on attachment 50967 [details] Proposed Patch Looks good.

Jian Li

Comment 3 2010-03-18 18:06:41 PDT

Committed as http://trac.webkit.org/changeset/56205.

Note You need to log in before you can comment on or make changes to this bug.