diff options
| author | jshin@chromium.org <jshin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-19 22:44:45 +0000 |
|---|---|---|
| committer | jshin@chromium.org <jshin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-19 22:44:45 +0000 |
| commit | a4d9ac41050e128f193b9ba7582e2f71e1797e8b (patch) | |
| tree | fc3c5453c60fe312638043603e4fbecf9065cb72 /DEPS | |
| parent | 66da08b94cbf0d93fbaff2233d8c358d61f6c088 (diff) | |
| download | chromium_src-a4d9ac41050e128f193b9ba7582e2f71e1797e8b.zip chromium_src-a4d9ac41050e128f193b9ba7582e2f71e1797e8b.tar.gz chromium_src-a4d9ac41050e128f193b9ba7582e2f71e1797e8b.tar.bz2 | |
Pull in googleurl @ r=120 to fix the query part encoding handling (the patch by Roland Steiner).
What Roland wrote at http://codereview.chromium.org/243028/show
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=8912,20507
TBR=brettw
Review URL: http://codereview.chromium.org/297010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29470 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'DEPS')
| -rw-r--r-- | DEPS | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -13,7 +13,7 @@ deps = { "/trunk/deps/support@20411", "src/googleurl": - "http://google-url.googlecode.com/svn/trunk@119", + "http://google-url.googlecode.com/svn/trunk@120", "src/sdch/open-vcdiff": "http://open-vcdiff.googlecode.com/svn/trunk@28", |