Bug 44072

Summary:

Null CFURLRef returned from provisionalURL after searching on zillow.com

Product:

WebKit

Reporter:

Sam Weinig <sam>

Component:

WebKit2

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

OS X 10.5

Attachments:

Description	Flags
Test case	none
Patch	barraclough: review+

Sam Weinig

Reported 2010-08-16 14:06:44 PDT

Created attachment 64519 [details] Test case Null CFURLRef returned from provisionalURL after searching on zillow.com. Zillo URL: http://show.zillow.com/delivery/afr.php?cb=5316409194&ixh=143&iw=130&sr=/vstatic/54f7169aa674ce4149bbce32ec27dddc&n=2&sc=1&ezw=300&fbcids=campaignid:4,campaignid:5&ezbh=200&ctd=/trk/ClkTrk.htm&ixw=286&v=1&csd=/static/css/showcase.css&ofid=fsearch_ResurrectionSearchPage__showcase_render_ShowcaseAdLink&ih=65&hst=http://www.zillow.com&r=13271&what=r13271sc1n2v1|campaignid:4,campaignid:5&rfl=http://www.zillow.com/ads/IframeSizer.htm&jsd=/static/js/showcase-concat.js&sld=/showcase/lead/SubmitLead.htm&csl=http://www.zillowstatic.com/vstatic/54f7169aa674ce4149bbce32ec27dddc/static/css/showcase.css&ezh=100&rad=/ads/ShowcaseAdIframeSizer.htm

Attachments
Test case (52 bytes, text/html) 2010-08-16 14:06 PDT, Sam Weinig	no flags	Details
Patch (2.64 KB, patch) 2010-08-16 14:07 PDT, Sam Weinig	barraclough: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Sam Weinig

Comment 1 2010-08-16 14:07:19 PDT

Created attachment 64520 [details] Patch

Gavin Barraclough

Comment 2 2010-08-16 15:21:17 PDT

Comment on attachment 64520 [details] Patch faulure spelling faulure.

Sam Weinig

Comment 3 2010-08-16 15:22:57 PDT

Landed in r65459.

Darin Adler

Comment 4 2010-08-16 17:52:38 PDT

Comment on attachment 64520 [details] Patch > + // This pattern of using utf8 and then falling back to latin1 on faulure matches KURL::createCFString with the > + // major differnce that KURL does not do a utf8 conversion and instead chops off the high bits of the utf16 character Typo: "differnce".

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