Bug 46434

Summary:

DOMWindow::revokeBlobURL erroneously uses ParsedURLString

Product:

WebKit

Reporter:

Alexey Proskuryakov <ap>

Component:

DOM

Assignee:

Jian Li <jianli>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

jianli

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Bug Depends on:

Bug Blocks:

42959

Attachments:

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

Description Alexey Proskuryakov 2010-09-23 17:24:06 PDT

The argument comes from the caller, there is absolutely no guarantee that it's in the form KURL expects.

This causes assertion failures whenever it's not (e.g. revokeBlobURL("[foo bar]");

For absolute URLs, we should just pass a null base: KURL(KURL(), string). But if it can be relative, a proper base should be passed.

Comment 1 Jian Li 2010-09-23 18:20:50 PDT

Created attachment 68635 [details]
Proposed Patch

Comment 2 Jian Li 2010-09-24 11:04:00 PDT

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