Make it possible to build url/ without ICU on android.

url/BUILD.gn

 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 component("url") {
   if (is_win) {
     # Don't conflict with Windows' "url.dll".
     output_name = "url_lib"
   }
   sources = [
+    "android/url_jni_registrar.cc",

[mef, 2014/05/01 19:45:15]

should this go into some android-specific section?

[mmenke, 2014/05/01 21:01:33]

Files that have platform names in their paths aren't compiled on other
platforms, so it's not necessary - you can find cases where this is done, and
cases where it isn't, some I'm not sure what the preferred style is.

For what it's worth, net/net.gypi does not do that.

[mef, 2014/05/02 15:10:48]

sgtm. I thought it only applied to name suffix, nice to know.

+    "android/url_jni_registrar.h",
     "gurl.cc",
     "gurl.h",
     "third_party/mozilla/url_parse.cc",
     "third_party/mozilla/url_parse.h",
     "url_canon.h",
     "url_canon_etc.cc",
     "url_canon_filesystemurl.cc",
     "url_canon_fileurl.cc",
     "url_canon_host.cc",
     "url_canon_icu.cc",
@@ skipping 24 matching lines @@
   # if (is_win) {
   #   TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
   #   'msvs_disabled_warnings': [ 4267, ]
   # }
 
   deps = [
     "//base",
     "//third_party/icu:icudata",
     "//third_party/icu",
   ]
+
+  if (use_icu_alternatives) {
+    sources -= [
+      "url_canon_icu.cc",
+      "url_canon_icu.h",
+    ]
+    deps -= [
+      "//third_party/icu:icudata",
+      "//third_party/icu",
+    ]
+
+    if (is_android) {
+      sources += [
+        "url_canon_icu_alternatives_android.cc",
+        "url_canon_icu_alternatives_android.h",
+      ]
+    }
+  }
 }
-
+  
 # TODO(dpranke): crbug.com/360936. Get this to build and run on Android.
 if (!is_android) {
   test("url_unittests") {
     sources = [
       "gurl_unittest.cc",
+      "url_canon_icu_unittest.cc",
       "url_canon_unittest.cc",
       "url_parse_unittest.cc",
       "url_test_utils.h",
       "url_util_unittest.cc",
     ]
 
     #if (is_posix && !is_mac && !is_ios) {
     #  # TODO(dmikurube): Kill linux_use_tcmalloc. http://crbug.com/345554
     #  if ((use_allocator!="none" && use_allocator!="see_use_tcmalloc") || (use_allocator=="see_use_tcmalloc" && linux_use_tcmalloc) {
     #    deps += "//base/allocator"
     #  }
     #}
 
     # if (is_win) {
     #   TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
     #   'msvs_disabled_warnings': [ 4267, ]
     # }
 
     deps = [
       ":url",
       "//base:i18n",
       "//base/test:run_all_unittests",
       "//testing/gtest",
       "//third_party/icu:icuuc",
     ]
+
+    if (use_icu_alternatives) {
+      sources -= [
+        "url_canon_icu_unittest.cc",
+      ]
+      deps -= [
+        "//base:i18n",
+        "//third_party/icu:icuuc",
+      ]
+  
+      if (is_android) {
+        sources += [
+          "url_canon_icu_alternatives_android.cc",
+        ]
+      }
+    }
   }
 }