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",
+    "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) {

[brettw, 2014/05/02 23:51:02]

This will break the GN build since this value is never defined. We should answer
the question I wrote in the GYP file and then I it will tell us what to do here.

[mmenke, 2014/05/03 01:15:50]

I hadn't realized gn wasn't yet being used or tested on any platform.

+    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 -= [
+        "//third_party/icu:icuuc",
+      ]
+    }
   }
 }