Fix encoding of the query part of an URL [Issue 20507].

Fix encoding of the query part of an URL.

Encoding of the query part of an URL was inconsistent: 
If it was part of a larger relative URL, it would get encoded. 
However, if the query part was stand-alone, then it would not be encoded.

As indicated by the original layout test referenced in the bug (see below), 
this patch changes the code so that the query part is encoded in both cases.

Note, however, that with this patch the layout test will NOT yet fully pass, 
since it also handles the fragment part differently. 
As the different handling of the fragment part seems to be quite on purpose, 
I have not modified the behavior - see the discussion at

    http://code.google.com/p/chromium/issues/detail?id=20507 .


BUG=20507, 8912
TEST=LayoutTests/http/tests/uri/resolve-encoding-relative.html

[Roland, 2009-09-28 14:11:33]

Hi, I uploaded a small patch that fixes an issue with URL encoding (query part),
as tested for in layout test 

LayoutTests/http/tests/uri/resolve-encoding-relative.html

Cheers,

Roland

[brettw, 2009-09-28 17:05:10]

LGTM. Would you prefer me to check this current patch in to googleurl now, or
wait for the next patch?

[jungshik at Google, 2009-09-28 18:42:10]

lgtm

[Roland, 2009-09-29 01:50:03]

On 2009/09/28 17:05:10, brettw wrote:
> LGTM. Would you prefer me to check this current patch in to googleurl now, or
> wait for the next patch?

Thanks - I'm fine either way, but since it's only a small patch, checking in
right away may be a bit more convenient.

[jungshik at Google, 2009-10-19 17:00:30]

On 2009/09/29 01:50:03, Roland wrote:
> On 2009/09/28 17:05:10, brettw wrote:
> > LGTM. Would you prefer me to check this current patch in to googleurl now,
or
> > wait for the next patch?
> 
> Thanks - I'm fine either way, but since it's only a small patch, checking in
> right away may be a bit more convenient.

Brett, have you checked this into googleurl?  It seems that this CL will fix
http://crbug.com/8912 as well.

[brettw, 2009-10-19 17:21:10]

Sorry, I lost this. I checked it in as googleurl@120. I did not pull DEPS.