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

net/http/http_auth_handler_digest.cc

Index: net/http/http_auth_handler_digest.cc
===================================================================
--- net/http/http_auth_handler_digest.cc	(revision 266199)
+++ net/http/http_auth_handler_digest.cc	(working copy)
@@ -6,7 +6,6 @@
 
 #include <string>
 
-#include "base/i18n/icu_string_conversions.h"
 #include "base/logging.h"
 #include "base/md5.h"
 #include "base/rand_util.h"
@@ -14,6 +13,7 @@
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "net/base/net_errors.h"
+#include "net/base/net_string_util.h"
 #include "net/base/net_util.h"
 #include "net/http/http_auth.h"
 #include "net/http/http_auth_challenge_tokenizer.h"
@@ -227,7 +227,7 @@
                                                    const std::string& value) {
   if (LowerCaseEqualsASCII(name, "realm")) {
     std::string realm;
-    if (!base::ConvertToUtf8AndNormalize(value, base::kCodepageLatin1, &realm))
+    if (!net::ConvertLatin1ToUtf8AndNormalize(value, &realm))

[mef, 2014/04/28 19:19:51]

Are we doing Latin1-specific function to avoid using base::kCodepageLatin1?

[mmenke, 2014/04/28 19:29:18]

Yes - it's declared in base/i18n, which is part of a project that depends on
icu, so can't use it without the icu dependency.  It's actually the same string
in Java as well, so we could make another net-specific version of it, but I
think that's ugly.

       return false;
     realm_ = realm;
     original_realm_ = value;