Consolidate most of net/'s icu_string_conversions.h dependencies.

net/DEPS

Index: net/DEPS
===================================================================
--- net/DEPS	(revision 266199)
+++ net/DEPS	(working copy)
@@ -23,7 +23,7 @@
 
   # Within net, only used by file: requests.
   "filename_util\.cc": [
-    "+base/i18n",
+    "+base/i18n/file_util_icu.h",
   ],
   
   # Functions largely not used by the rest of net.
@@ -37,26 +37,17 @@
     "+base/i18n",
   ],
 
-  # Only use icu for string conversions.
-  "escape_unittest\.cc": [
+  # Consolidated string functions that depend on icu.
+  "net_string_util_icu\.cc": [
+    "+base/i18n/i18n_constants.h",
     "+base/i18n/icu_string_conversions.h",
+    "+third_party/icu/source/common/unicode/ucnv.h"
   ],
-  "http_auth_handler_basic\.cc": [
-    "+base/i18n/icu_string_conversions.h",
-  ],
-  "http_auth_handler_digest\.cc": [
-    "+base/i18n/icu_string_conversions.h",
-  ],
+
+  # Only use icu for string conversions.
   "proxy_script_fetcher_impl\.cc": [
     "+base/i18n/icu_string_conversions.h",
   ],

[mmenke, 2014/04/28 19:01:37]

Not sure if we need to do this one.  It also uses a slightly different behavior
on error (Add special character).  Can do that in Java, but not sure we want a
function with another behavior unless we need it.

-  "x509_cert_types_mac\.cc": [
-    "+base/i18n/icu_string_conversions.h",
-  ],
-  "http_content_disposition\.cc": [
-    "+base/i18n/icu_string_conversions.h",
-    "+third_party/icu",
-  ],
   "websocket_channel\.h": [
     "+base/i18n",
   ],