Bug 24580

Summary: KURLGoogle differs from KURL with treatment of query string
Product: WebKit Reporter: Mike Belshe <mbelshe>
Component: WebCore Misc.Assignee: Mike Belshe <mbelshe>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Attachments:
Description Flags
patch
dglazkov: review+
updated patch dglazkov: review+

Description Mike Belshe 2009-03-13 11:38:19 PDT 
KURL does not include the ?, but KURLGoogle does.  I believe this is because code has changed.

Make KURLGoogle match KURL.
Comment 1 Mike Belshe 2009-03-13 11:53:32 PDT 
Created attachment 28588 [details]
patch
Comment 2 Dimitri Glazkov (Google) 2009-03-13 13:22:52 PDT 
Comment on attachment 28588 [details]
patch

Yay, good!
Comment 3 Dimitri Glazkov (Google) 2009-03-13 15:43:05 PDT 
Landed as http://trac.webkit.org/changeset/41689.
Comment 4 Mike Belshe 2009-03-13 16:16:50 PDT 
Created attachment 28609 [details]
updated patch
Comment 5 Mike Belshe 2009-03-13 16:17:44 PDT 
Patch the patch to actually work.
Comment 6 Dimitri Glazkov (Google) 2009-03-16 09:13:11 PDT 
Comment on attachment 28609 [details]
updated patch

Yikes.
Comment 7 Dimitri Glazkov (Google) 2009-03-16 09:13:51 PDT 
Compile fix landed as http://trac.webkit.org/changeset/41733.