Bug 24580 - KURLGoogle differs from KURL with treatment of query string
Summary: KURLGoogle differs from KURL with treatment of query string
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P2 Normal
Assignee: Mike Belshe
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-03-13 11:38 PDT by Mike Belshe
Modified: 2009-03-16 09:13 PDT (History)
0 users

See Also:


Attachments
patch (1.37 KB, patch)
2009-03-13 11:53 PDT, Mike Belshe
dglazkov: review+
Details | Formatted Diff | Diff
updated patch (1.38 KB, patch)
2009-03-13 16:16 PDT, Mike Belshe
dglazkov: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.