Forward includes to url/

This is in preparation for moving the files to the Chromium repository. The
current headers will forward Chromium clients to files in the new location so
the include paths can be updated in a separate pass.

Review https://codereview.chromium.org/14090005/
Patch by Thiago Farina

git-svn-id: http://google-url.googlecode.com/svn/trunk@185 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix C++11 compilation on iOS

Fixes an "invalid suffix on literal" warning.

BUG=google-url:34
Patch by stuartmorgan@chromium.org
Original review https://codereview.chromium.org/13467017/

git-svn-id: http://google-url.googlecode.com/svn/trunk@184 8873c55e-713a-0410-88f8-23d9c3d90b1b

Typo fix: we're referring to the wrong spec_.

BUG=164643
Review URL: https://codereview.chromium.org/11821041

git-svn-id: http://google-url.googlecode.com/svn/trunk@183 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add googleurl.gyp to make it possible to build using gyp tool.

Patch by tfarina@chromium.org
Original review https://codereview.chromium.org/12045039/

git-svn-id: http://google-url.googlecode.com/svn/trunk@182 8873c55e-713a-0410-88f8-23d9c3d90b1b

Make ResolveRelative work with all hierarchical URLs.

This makes the ResolveRelative method work correctly for all
hierarchical URLs while preserving the special support for the
"standard" schemes.

The change is needed for Android WebView, where it is perfectly
valid for an embedder to load a data URL with a base URL that is
a custom scheme (like myapp://resources/) and then later intercept
those requests.

Review=https://codereview.chromium.org/11367010/
BUG=http://crbug.com/159832
Patch by Martin Kosiba

git-svn-id: http://google-url.googlecode.com/svn/trunk@181 8873c55e-713a-0410-88f8-23d9c3d90b1b

SchemeIsSecure must return true for "wss"

BUG=http://crbug.com/155265
Patch by toyoshim@chromium.org
Review https://codereview.chromium.org/11092076/

git-svn-id: http://google-url.googlecode.com/svn/trunk@180 8873c55e-713a-0410-88f8-23d9c3d90b1b

Expose functions to stringify IPv4 and IPv6 addresses

BUG=http://crbug.com/126212

Code review: https://chromiumcodereview.appspot.com/10919114/

git-svn-id: http://google-url.googlecode.com/svn/trunk@179 8873c55e-713a-0410-88f8-23d9c3d90b1b

Revert r176, this breaks some Chrome tests.

git-svn-id: http://google-url.googlecode.com/svn/trunk@178 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add a missing include.

git-svn-id: http://google-url.googlecode.com/svn/trunk@177 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix handling of URLs with no schema.

BUG=google-url:29
TEST=GURLTest::NonURL
Patch by mnaganov@chromium.org
Review https://chromiumcodereview.appspot.com/10542140/

git-svn-id: http://google-url.googlecode.com/svn/trunk@176 8873c55e-713a-0410-88f8-23d9c3d90b1b

Change #include <memory.h> to #include <string.h>

This allows the file to build with the NaCl toolchain.

BUG=none
TEST=none
Review = http://codereview.chromium.org/10389101/
Author = Bill Budge

git-svn-id: http://google-url.googlecode.com/svn/trunk@175 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix path component length for zero-length paths canonicalized to '/'.
Review URL: https://chromiumcodereview.appspot.com/10386035

git-svn-id: http://google-url.googlecode.com/svn/trunk@174 8873c55e-713a-0410-88f8-23d9c3d90b1b

Remove ifdef guard for FULL_FILESYSTEM_URL_SUPPORT.
Add creation of inner_url_ to the places outside the GURL constructor that
effectively construct GURLs.

git-svn-id: http://google-url.googlecode.com/svn/trunk@173 8873c55e-713a-0410-88f8-23d9c3d90b1b

Remove a test that only works on one of windows/*nix; it's not needed, and not
worth #ifdeffing to have two different expectations.

Review: https://chromiumcodereview.appspot.com/9961021/
Patch by ericu@chromium.org

git-svn-id: http://google-url.googlecode.com/svn/trunk@172 8873c55e-713a-0410-88f8-23d9c3d90b1b

I fixed the expectation [the URL is invalid] and added a few slashes to make it
more obvious how we canonicalize bad filesystem:file URLs that are missing the
storage type specifier.

BUG=http://crbug.com/121671
Review https://chromiumcodereview.appspot.com/9956109/
Patch by ericu@chromium.org

git-svn-id: http://google-url.googlecode.com/svn/trunk@171 8873c55e-713a-0410-88f8-23d9c3d90b1b

Disable failing filesystem URL test.

git-svn-id: http://google-url.googlecode.com/svn/trunk@170 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix a memory leak; properly initialize inner_parsed before use.

TEST=none
BUG=none

Patch by ericu@chromium.org
Review: https://chromiumcodereview.appspot.com/9838097/

git-svn-id: http://google-url.googlecode.com/svn/trunk@169 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add missing file from r166.

Add support for real full filesystem URL parsing and canonicalization.

See also the chromium [http://codereview.chromium.org/7811006/] and webkit
[http://codereview.chromium.org/8856006/] changes that will depend on this.

Review:http://codereview.appspot.com/4961060/
Patch by ericu@chromium.org

git-svn-id: http://google-url.googlecode.com/svn/trunk@168 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix the GURL change [compile error, two small changes to make things safe while
the rest of the changes aren't in yet].

Review = http://codereview.chromium.org/8974033/
Patch by ericu@chromium.org

git-svn-id: http://google-url.googlecode.com/svn/trunk@167 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add support for real full filesystem URL parsing and canonicalization.

See also the chromium [http://codereview.chromium.org/7811006/] and webkit
[http://codereview.chromium.org/8856006/] changes that will depend on this.

Review:http://codereview.appspot.com/4961060/
Patch by ericu@chromium.org

git-svn-id: http://google-url.googlecode.com/svn/trunk@166 8873c55e-713a-0410-88f8-23d9c3d90b1b

Remove a static initializer from gurl.cc when using libstdc++.

Patch by Nico Weber

BUG=http://code.google.com/p/chromium/issues/detail?id=94794
Review=http://codereview.chromium.org/9035011/

git-svn-id: http://google-url.googlecode.com/svn/trunk@165 8873c55e-713a-0410-88f8-23d9c3d90b1b

GURL: Export some missing symbols needed to run gurl
unit tests on Linux.

BUG=http://crbug.com/102738
TEST=none

Patch contributed by rvargas@chromium.org

Code review: http://codereview.chromium.org/8508018/

git-svn-id: http://google-url.googlecode.com/svn/trunk@162 8873c55e-713a-0410-88f8-23d9c3d90b1b

GURL: Export a few functions used only by gurl unittests.

Even though the new exports are of "internal" stuff, they
are being used directly by unit tests, so the components
build is not building gurl uittests at all.

We want to make this the default configuration, so all tests
have to run.

BUG=102738
TEST=none

Patch contributed by rvargas@chromium.org.

Code review: http://codereview.chromium.org/8474013

git-svn-id: http://google-url.googlecode.com/svn/trunk@161 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add a codereview.settings file.

Previously, gcl had default settings to use the chromium's codereview site by
default. These default settings were removed and now need to be made explicit.

git-svn-id: http://google-url.googlecode.com/svn/trunk@160 8873c55e-713a-0410-88f8-23d9c3d90b1b

Provide a way to get the parsed IP address out of CanonicalizeHostVerbose.

BUG=http://code.google.com/p/google-url/issues/detail?id=25
Patch by joth@chromium.org

git-svn-id: http://google-url.googlecode.com/svn/trunk@159 8873c55e-713a-0410-88f8-23d9c3d90b1b

Avoid indexing into a string when it's empty to avoid an assertion failure
when using _GLIBCXX_DEBUG.

Patch by tzik@chromium.org
Original review http://codereview.chromium.org/7198018/

git-svn-id: http://google-url.googlecode.com/svn/trunk@158 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add a function to URL-encode a path component, using the same rules as
encodeURIComponenet. This is needed by the filesystem API to fix*
http://code.google.com/p/chromium/issues/detail?id=78860, although the full f>
will require more code than this.

BUG=http://crbug.com/78860
TEST=unit test
Original review=http://codereview.chromium.org/7046062/
Patch by ericu@chromium.org

git-svn-id: http://google-url.googlecode.com/svn/trunk@157 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix handling of certain URLs with long runs of numbers.

BUG=84077
TEST=URLCanonTest.Host

Code review: http://codereview.chromium.org/7020030/

Patch contibuted by mmenke@chromium.org

git-svn-id: http://google-url.googlecode.com/svn/trunk@156 8873c55e-713a-0410-88f8-23d9c3d90b1b

url_canon_ip: For IPv6 literals with an IPv4-style suffix, remove the
restriction that the prefix must match ::/96 or ::ffff:0:0/96.

People have been complaining that the URL parser won't accept their
NAT64 prefixes, and I can't see any obvious reason for this
restriction to exist.

The code I'm deleting was originally added here:
http://codereview.chromium.org/99183

Review: http://codereview.chromium.org/6902189/

git-svn-id: http://google-url.googlecode.com/svn/trunk@155 8873c55e-713a-0410-88f8-23d9c3d90b1b

Update GURL_API to cover non-Windows platforms.

When GURL_DLL is set, we export any symbols tagged GURL_API.

Patch by Evan Martin <evan@chromium.org>

git-svn-id: http://google-url.googlecode.com/svn/trunk@154 8873c55e-713a-0410-88f8-23d9c3d90b1b

Change gurl test main to look for both icudt.dll and icudtXX.dll

In Chrome, we dropped a version number (e.g. '46') from the name of icu data dll
(see http://codereview.chromium.org/6625032/ ), but
for 3rd party embedders, this CL makes gurl test main look for both files.

BUG=chromium:74076 (http://crbug.com/74076)
TEST=gurl_test runs fine whether icudt has a version number or not.
Original review=http://codereview.chromium.org/6623060/

git-svn-id: http://google-url.googlecode.com/svn/trunk@153 8873c55e-713a-0410-88f8-23d9c3d90b1b

Check that gtest has death test support before trying to compile a test using
it (avoiding compile failure).

Patch by Thomas Van Lenten <thomasvl@chromium.org>

BUG=none
TEST=none

git-svn-id: http://google-url.googlecode.com/svn/trunk@152 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix api export for iostream operator <<.

git-svn-id: http://google-url.googlecode.com/svn/trunk@151 8873c55e-713a-0410-88f8-23d9c3d90b1b

Make the GURL output stream operator non-inline. Apparently some compilers need
more than just ios_fwd to compile the implementation, and we don't want to add
this dependency for all the files that include gurl.h.

TEST=it compiles
BUG=none

git-svn-id: http://google-url.googlecode.com/svn/trunk@150 8873c55e-713a-0410-88f8-23d9c3d90b1b

Remove the explicit destructor from GURL. This was causing weird problems with
linking some Chrome modules.

git-svn-id: http://google-url.googlecode.com/svn/trunk@149 8873c55e-713a-0410-88f8-23d9c3d90b1b

Move constructors to the .cc files to prevent inline code explosion.

Patch by erg@chromium.org

git-svn-id: http://google-url.googlecode.com/svn/trunk@148 8873c55e-713a-0410-88f8-23d9c3d90b1b

Remove a semicolon as a host terminator. This was originally added to handle
the fact that a semicolon is a "parameter" in the path of the URL. We removed
special parsing for these long ago since they aren't used in practice.

This will now treat the input foo;bar as a single hostname with an invalid
character in the middle, rather than foo/;bar.

git-svn-id: http://google-url.googlecode.com/svn/trunk@147 8873c55e-713a-0410-88f8-23d9c3d90b1b

Remove unused static function to avoid compiler warning.

git-svn-id: http://google-url.googlecode.com/svn/trunk@146 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix a typo in a comment about scheme handling.

Patch by Ilya Sherman.

git-svn-id: http://google-url.googlecode.com/svn/trunk@145 8873c55e-713a-0410-88f8-23d9c3d90b1b

Remove redundant definition for operator<<

git-svn-id: http://google-url.googlecode.com/svn/trunk@144 8873c55e-713a-0410-88f8-23d9c3d90b1b

Remove redundant check for failure to read a unicode char. ReadUTFChar already checks for valitity and returns the replacement character on failure.

TEST=none
BUG=none
Review URL: http://codereview.chromium.org/3048014

git-svn-id: http://google-url.googlecode.com/svn/trunk@143 8873c55e-713a-0410-88f8-23d9c3d90b1b

Coverity: Fix DoAppendUTF8 handling of overly large code points.

CID=220
TEST=URLCanonTest::DoAppendUTF8
BUG=none
Patch by mattm@chromium.org
Original review: http://codereview.chromium.org/2877015

git-svn-id: http://google-url.googlecode.com/svn/trunk@142 8873c55e-713a-0410-88f8-23d9c3d90b1b

Change <ostream> to <iosfwd> to further reduce header file dependencies.

git-svn-id: http://google-url.googlecode.com/svn/trunk@141 8873c55e-713a-0410-88f8-23d9c3d90b1b

Use ostream instead of iostream to avoid statics brought in by iostream. It should also be a little smaller and make compiles faster.

git-svn-id: http://google-url.googlecode.com/svn/trunk@140 8873c55e-713a-0410-88f8-23d9c3d90b1b

-. Check GURL_IMPLEMENTATION value for dll build so it is consistent with WEBKIT_IMPLEMENTATION.
-. Fix GURL_API for url_parse.

R=brettw
TEST=none
BUG=none

Review URL: http://codereview.chromium.org/2900011

git-svn-id: http://google-url.googlecode.com/svn/trunk@139 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add option to build googleurl as dll
-. Export/import google url APIs if GURL_DLL is defined.
-. Move DecodeURLEscapeSequences from webcore KURLGoogle
   to url_util to avoid external code accessing googleurl
   global varaibles.

R=brettw
BUG=46311
TEST=Googleurl_unittests

Review URL: http://codereview.chromium.org/2939004

git-svn-id: http://google-url.googlecode.com/svn/trunk@138 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix comment about threadsafety.

git-svn-id: http://google-url.googlecode.com/svn/trunk@137 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add a missing "const" keyword so the wide version has the same signature as the narrow version (more usable by templatized functions that way).

Code Review: http://codereview.chromium.org/2815020

git-svn-id: http://google-url.googlecode.com/svn/trunk@136 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix a comment typo (the description of "compat" and "mapped" IPv6 addresses was reversed).

Review URL: http://codereview.chromium.org/2859009/show

git-svn-id: http://google-url.googlecode.com/svn/trunk@135 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix a crash in IP address checking because the boundary case for an empty or invalid IP address isn't handled correctly and we can go reading random memory. This doesn't happen during normal host canonicalization because it won't try to canonicalize the IP address of an empty host. It only happens if the app is trying to see if a previously-canonicalized URL is an IP.

BUG=http://code.google.com/p/chromium/issues/detail?id=43819
TEST=none. I wrote a simple unit tests but it doesn't actually work to catch this case. I could not figure out how to get the old code to fail reproducibly, since it starts reading random memory until it finds a character that's invalid. Normally this happens quickly so the crash doesn't occur. It only happens if all characters in memory following the string are valid IP address characters.
Review URL: http://codereview.chromium.org/2027007

git-svn-id: http://google-url.googlecode.com/svn/trunk@134 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add the ability to prevent future changes to the standard scheme list
to prevent threading errors like we've seen in Chrome.
Review URL: http://codereview.chromium.org/1607014

git-svn-id: http://google-url.googlecode.com/svn/trunk@131 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add a newline to the end of url_util_unittest to make it compile on Mac.

git-svn-id: http://google-url.googlecode.com/svn/trunk@130 8873c55e-713a-0410-88f8-23d9c3d90b1b

Canonicalize URL before using Extractscheme in DoFindAndCompareScheme function.

Add some additional clarifying comments for scheme finding and canonicalization
(the code is correct, this is just to make the current behavior more explicit
and hopefully prevent people breaking it).

BUG=http://crbug.com/40445
TEST=URLUtilTest.FindAndCompareScheme

git-svn-id: http://google-url.googlecode.com/svn/trunk@129 8873c55e-713a-0410-88f8-23d9c3d90b1b

Relocate a function so that we don't need any forward declarations.

Patch by Evan Martin
Original review: http://codereview.chromium.org/650139

git-svn-id: http://google-url.googlecode.com/svn/trunk@128 8873c55e-713a-0410-88f8-23d9c3d90b1b

Pass @ through without escaping or unescaping. This is compatible with IE8 and seems to fix some sites.

BUG=http://crbug.com/23933
TEST=included unit test
Review URL: http://codereview.chromium.org/1556014

git-svn-id: http://google-url.googlecode.com/svn/trunk@127 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add an initialize and a cleanup function for projects wishing to use googleurl
in a multithreaded environment (who aren't already using AddStandardScheme) or
in a DLL.

BUG=17

git-svn-id: http://google-url.googlecode.com/svn/trunk@126 8873c55e-713a-0410-88f8-23d9c3d90b1b

actually define the function I added in the previous change for backwards API compat.

git-svn-id: http://google-url.googlecode.com/svn/trunk@125 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add a backwards-compat workaround to make landing the previous change in Chrome and WebKit easier.

git-svn-id: http://google-url.googlecode.com/svn/trunk@124 8873c55e-713a-0410-88f8-23d9c3d90b1b

Remove the rule that "://" means a standard URL. This fixes a number of bugs
and layout tests. The only thing we use now to determine whether a scheme is
standard scheme is whether it is on a known list.

Fix some bugs around setting protocols. Previously, we wouldn't reparse the
entire URL, which would give weird results in some cases, since which part
actually gets replaced changes if you change the protocol. This change has us
replace the protocol, then completely reparse the URL which ensures we don't
miss any edge cases. It is a bit slower, but changing the protocol is very
unusual.

This differs from KURL in several cases. On the
fast/dom/HTMLAnchorElement/set-href-attribute-hostname.html
layout test, we fail setting a hostname on a URL with "foo" protocol because we
no longer treat a "foo" URL as a standard protocol. However, this matches
Firefox's behavior, so I think there should not be compat problems.

Other differences in that test are that we allow setting hosts with spaces in
them for IE compat, and that when we generate the URL "c:/path/testurl.html" on
Windows, we reinterpret that as a file URL to file:///C:/path/testurl.html.
This is how "c:/path/testurl.html" would be interpreted if we just found it on
a page, so I think this behavior is correct, even though no other browser does
this (our handling of these generally matches IE, but it fails to set a "c"
protocol with an exception in this case).

http://crbug.com/160
Review URL: http://codereview.chromium.org/564011

git-svn-id: http://google-url.googlecode.com/svn/trunk@123 8873c55e-713a-0410-88f8-23d9c3d90b1b

When resolving URL, empty relative URL should remove ref component.

R=brettw
BUG=16
TEST=none
Patch by dglazkov@chromium.org
Original review http://codereview.chromium.org/600087

git-svn-id: http://google-url.googlecode.com/svn/trunk@122 8873c55e-713a-0410-88f8-23d9c3d90b1b

Use size_t for string lengths to avoid compiler warnings.

Patch by Gregory Dardyk

git-svn-id: http://google-url.googlecode.com/svn/trunk@121 8873c55e-713a-0410-88f8-23d9c3d90b1b

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
Patch by Roland Steiner, review URL http://codereview.chromium.org/243028

git-svn-id: http://google-url.googlecode.com/svn/trunk@120 8873c55e-713a-0410-88f8-23d9c3d90b1b

Remove ICU dependency from url_canon_internal. We were just picking up some
types and MAX* values for those tyeps from ICU, and weren't using it for
anything else in this context.

Patch by Matthew Steele

git-svn-id: http://google-url.googlecode.com/svn/trunk@119 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix off-by-one in range check.

http://crbug.com/5490
Patch by mattm@chromium.org
Review: http://codereview.chromium.org/216020

git-svn-id: http://google-url.googlecode.com/svn/trunk@118 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix googleurl to use icu version-agnostic directory.

Adds text files as item in the projects.
Review URL: http://codereview.chromium.org/214013

git-svn-id: http://google-url.googlecode.com/svn/trunk@117 8873c55e-713a-0410-88f8-23d9c3d90b1b

Change WebSocket default port

The Web Socket protocol specification draft has changed the default port of Web Socket recently.
http://tools.ietf.org/rfcdiff?difftype=--hwdiff&url2=draft-hixie-thewebsocket...

BUG=none
TEST=googleurl_unittests URLCanonTest.CanonicalizeStandardURL passes.

Patch contributed by ukai@chromium.org.

git-svn-id: http://google-url.googlecode.com/svn/trunk@116 8873c55e-713a-0410-88f8-23d9c3d90b1b

Update the unit tests to allow a colon in UNC hostnames on Windows. This test
was broken in r107 but we weren't running the tests properly so never noticed.

Colon is probably not allowed by Windows, but its the "natural" result of the
way the canonicalizer is written. There doesn't seem to be a strong argument
for why allowing it here would be bad, so we just tolerate it and the load will
fail later.
Review URL: http://codereview.chromium.org/173535

git-svn-id: http://google-url.googlecode.com/svn/trunk@115 8873c55e-713a-0410-88f8-23d9c3d90b1b

Use U_ICU_VERSION_SHORT macro instead of hard-coding icu data dll and module
name.

BUG=8198
TEST=googleurl test should be built and run without a problem.
Patch by Jungshik Shin
Review: http://codereview.chromium.org/174265

git-svn-id: http://google-url.googlecode.com/svn/trunk@114 8873c55e-713a-0410-88f8-23d9c3d90b1b

All host names with nonascii characters (cyrillic) + escapable characters
(,)...  were shown corrupted in suggestion box in Omnibox2, because we escaped
punicode string after it was created. I added some more escaping before IDNA
conversion.

Unittest updated, and main method added so we can init ICU data properly.

http://crbug.com/5490
TEST=Follow bug description, and make sure that test is as expected (orignal
text+%28).

Patch by: cira@chromium.org
Review: http://codereview.chromium.org/160589

git-svn-id: http://google-url.googlecode.com/svn/trunk@113 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add Web Socket default port.

As described in http://tools.ietf.org/html/draft-hixie-thewebsocketprotocol,
ws (Web Socket) scheme uses port 81 by default, and
wss (secure Web Socket) scheme uses port 815 by default.

BUG=none
TEST=googleurl_unittests  URLCanonTest.CanonicalizeStandardURL passes.

Review URL: http://codereview.chromium.org/159401

git-svn-id: http://google-url.googlecode.com/svn/trunk@112 8873c55e-713a-0410-88f8-23d9c3d90b1b

Canonicalize '%' in userinfo properly.  It should not be escaped.
According to RFC 1738:
user           = *[ uchar | ";" | "?" | "&" | "=" ]
password       = *[ uchar | ";" | "?" | "&" | "=" ]
uchar          = unreserved | escape
escape         = "%" hex hex
BUG=http://code.google.com/p/google-url/issues/detail?id=10

Review URL: http://codereview.chromium.org/160282

git-svn-id: http://google-url.googlecode.com/svn/trunk@111 8873c55e-713a-0410-88f8-23d9c3d90b1b

Expose in url_canon_ip.h the previously internal-only functions:

 url_canon::IPv4AddressToNumber()
 url_canon::IPv6AddressToNumber()

Motivation:

I would like to use url_canon::IPv6AddressToNumber() in some chromium
network tests, to create mock IPv6 socketaddrs.

The system-provided equivalents (like getaddrinfo() or inet_ntop()) are
unreliable for this purpose, since they only work on IPv6-enabled
machines.

BUG=NONE
TEST=NONE
R=brettw

git-svn-id: http://google-url.googlecode.com/svn/trunk@110 8873c55e-713a-0410-88f8-23d9c3d90b1b

Require hosts for non-file URLs.
BUG=9
Patch by willchan@chromium.org
Original review: http://codereview.chromium.org/149588

git-svn-id: http://google-url.googlecode.com/svn/trunk@109 8873c55e-713a-0410-88f8-23d9c3d90b1b

Make DoCanonicalizeIPv6Address safer by changing loop condition and
adding a DCHECK

Patch by yuzo@chromium.org
Original review URL: http://codereview.chromium.org/155077

git-svn-id: http://google-url.googlecode.com/svn/trunk@108 8873c55e-713a-0410-88f8-23d9c3d90b1b

url_canon: New CanonicalizeHostVerbose() function.

This function allows the caller to distinguish between 4 cases:
- A valid, canonical IPv4 address.
- A valid, canonical IPv6 address.
- A "broken" address.  This is something which kinda almost looks like an IP
  address, but has been deemed to be invalid.  For IPv4, this indicates a
  numerical component which is longer than 16 digits, or one which can't be
  used without truncation/overflow.  For IPv6, this is any non-parseable
  address containing a colon or square brackets.
- "neutral".  This indicates that the output doesn't really look like an
  IP address at all, and should probably be treated as a hostname.

The CanonHostInfo struct is meant to be extensible.  In addition to the 4 cases
above, it also returns the number of components used to make an IPv4 address,
which will be useful to Chromium, and will make it possible stop exposing
FindIPv4Components.

This is the first step toward making Chrome behave better for these cases:
- http://[www.google.com]/
- http://192.168.0.0000000000000000000000000001/
- http://192.168.0.99999/

Edit:
- Changed the signature of CanonicalizeIPAddress(), instead of creating
  a new "Verbose" version.
- For completeness, now includes the CanonicalizeHost() changes,
  with an extended-output CanonicalizeHostVerbose(), because some callers
  only care whether the canonicalization was a success or not.

Code review:
http://codereview.chromium.org/114050

git-svn-id: http://google-url.googlecode.com/svn/trunk@107 8873c55e-713a-0410-88f8-23d9c3d90b1b

url_parse: Segment partially-typed IPv6 literals.

In Chromium, when a user is typing an IPv6 literal into the address bar, the
autocomplete system goes nuts, trying to grab a port from the end of the
address before it's complete.

Fix this by parsing unterminated IPv6 literals as if there were no port number.
These partial addresses will not survive canonicalization, but they keep the
autocompleter running smoothly as the address is typed.

Also, fix some minor lint warnings elsewhere in url_parse.cc

Code review:
http://codereview.chromium.org/115748

git-svn-id: http://google-url.googlecode.com/svn/trunk@106 8873c55e-713a-0410-88f8-23d9c3d90b1b

Move ICU dependent function ReadUTFChar into the icu.cc file.  This avoids icu
dependency in other object files that use inline functions from
url_canon_internal.h.

Patch by tommi@chromium.org.
Review: http://codereview.chromium.org/118062

git-svn-id: http://google-url.googlecode.com/svn/trunk@105 8873c55e-713a-0410-88f8-23d9c3d90b1b

Canonicalize IPv6 addresses.

BUG=http://code.google.com/p/google-url/issues/detail?id=2

Review URL: http://codereview.chromium.org/99183

git-svn-id: http://google-url.googlecode.com/svn/trunk@104 8873c55e-713a-0410-88f8-23d9c3d90b1b

Support Z as a first letter of a scheme.

Patch by Paul Marks.
Review URL: http://codereview.chromium.org/113514

git-svn-id: http://google-url.googlecode.com/svn/trunk@103 8873c55e-713a-0410-88f8-23d9c3d90b1b

A URL can contain an IPv6 address literal, like [2001:db8::2].  Users of
the GURL library sometimes need the square brackets, and sometimes don't.

Checked in for Paul Marks
Review URL: http://codereview.chromium.org/113187

git-svn-id: http://google-url.googlecode.com/svn/trunk@102 8873c55e-713a-0410-88f8-23d9c3d90b1b

Escape double quotes (U+0022).

BUG=http://code.google.com/p/chromium/issues/detail?id=7024
Review=http://codereview.chromium.org/62081
Checked in for eroman

git-svn-id: http://google-url.googlecode.com/svn/trunk@101 8873c55e-713a-0410-88f8-23d9c3d90b1b

Convert all uses of UTF16Char and UTF16String to char16 and string16
Review URL: http://codereview.chromium.org/42613

git-svn-id: http://google-url.googlecode.com/svn/trunk@100 8873c55e-713a-0410-88f8-23d9c3d90b1b

Eliminate dubious uses of basic_string<> in googleurl

Review URL: http://codereview.chromium.org/42588

git-svn-id: http://google-url.googlecode.com/svn/trunk@99 8873c55e-713a-0410-88f8-23d9c3d90b1b

Added missing <stdlib.h>, needed on ubuntu 8.10.
Patch by Dan Kegel.
http://codereview.chromium.org/46080

git-svn-id: http://google-url.googlecode.com/svn/trunk@98 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add basic PRESUBMIT script.
Review URL: http://codereview.chromium.org/39188

git-svn-id: http://google-url.googlecode.com/svn/trunk@97 8873c55e-713a-0410-88f8-23d9c3d90b1b

Make ParseAuthority part of the public API.
  http://codereview.chromium.org/28016

Patch by Eric Roman.

git-svn-id: http://google-url.googlecode.com/svn/trunk@96 8873c55e-713a-0410-88f8-23d9c3d90b1b

Support UNC-like file: URL parsing on non-Windows platforms.

Patch by Michael Moss:
http://codereview.chromium.org/18565/show

git-svn-id: http://google-url.googlecode.com/svn/trunk@95 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add the ability to get the effective port for schemes with a default port.

Patch by Eric Roman.

git-svn-id: http://google-url.googlecode.com/svn/trunk@94 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add basic IPv6 support. This just identifies possibly valid IPv6 addresses and
passes them through so they can be handled by the network layer if they are
supported there (previously we would reject all of them). This does not do
actual canonicalization yet.

Patch by Erik Kline.

git-svn-id: http://google-url.googlecode.com/svn/trunk@93 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix some unittest warnings under GCC.

git-svn-id: http://google-url.googlecode.com/svn/trunk@92 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix compile warnings on GCC.

git-svn-id: http://google-url.googlecode.com/svn/trunk@91 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix two build problems on Linux. The unit test problems are becuase the default
char there is unsigned. The UTF-8 reading problems fixes a warning when the
macro writes -1 to an unsigned value.

git-svn-id: http://google-url.googlecode.com/svn/trunk@90 8873c55e-713a-0410-88f8-23d9c3d90b1b

Make < and > be escaped in the query part of the URL.

BUG=1261537

git-svn-id: http://google-url.googlecode.com/svn/trunk@89 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix some Unix-related compile issues.
Use GTest instead of GUnit.

git-svn-id: http://google-url.googlecode.com/svn/trunk@88 8873c55e-713a-0410-88f8-23d9c3d90b1b

Expose the encoding parameter when resolving relative URLs in Resolve(). This
is temporary until we can figure out how this object will handle encodings
properly.

git-svn-id: http://google-url.googlecode.com/svn/trunk@87 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix a crash if we were calling ReplaceComponents on a URL with no scheme (such
as an empty URL). Add some unit tests around this.

git-svn-id: http://google-url.googlecode.com/svn/trunk@86 8873c55e-713a-0410-88f8-23d9c3d90b1b

Make relative URL resolving check the scheme for validity and fall back to
relative URLs if it's not.

git-svn-id: http://google-url.googlecode.com/svn/trunk@85 8873c55e-713a-0410-88f8-23d9c3d90b1b

Fix several limitations of the canonicalizer. This validates characters in the
reference fragment and user info sections. I did some cleanup associated with
this for character validity checking. This also makes the query escape certain
characters like # which could possibly get set if the query is replaced with
something new.

This is a major change to the host canonicalizer. We now support a third mode
for compatibility to allow more characters and to escape them if we see them
in hosts. Even though these are not valid DNS names, I have seen them being
used in third-party protocols that happen to be "standard" and most programs
support them.

git-svn-id: http://google-url.googlecode.com/svn/trunk@83 8873c55e-713a-0410-88f8-23d9c3d90b1b

Remove encoding for mailto URLs.

git-svn-id: http://google-url.googlecode.com/svn/trunk@81 8873c55e-713a-0410-88f8-23d9c3d90b1b

Add forgotten file in preivious mailto checkin.

git-svn-id: http://google-url.googlecode.com/svn/trunk@79 8873c55e-713a-0410-88f8-23d9c3d90b1b