Translate ONC ProxySettings <-> Shill ProxyConfig

chrome/browser/chromeos/ui_proxy_config.cc

 // Copyright 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.
 
 #include "chrome/browser/chromeos/ui_proxy_config.h"
 
 #include "base/logging.h"
 #include "base/values.h"
 #include "components/proxy_config/proxy_config_dictionary.h"
 #include "net/proxy/proxy_config.h"
+#include "url/url_constants.h"
+
+namespace {
+const char kSocksScheme[] = "socks";
+}
 
 namespace chromeos {
 
 UIProxyConfig::UIProxyConfig()
     : mode(MODE_DIRECT),
       state(ProxyPrefs::CONFIG_UNSET),
       user_modifiable(true) {
 }
 
 UIProxyConfig::~UIProxyConfig() {
@@ skipping 92 matching lines @@
     }
     case MODE_SINGLE_PROXY: {
       std::string spec;
       if (single_proxy.server.is_valid())
         spec = single_proxy.server.ToURI();
       return ProxyConfigDictionary::CreateFixedServers(
           spec, bypass_rules.ToString());
     }
     case MODE_PROXY_PER_SCHEME: {
       std::string spec;
-      EncodeAndAppendProxyServer("http", http_proxy.server, &spec);
-      EncodeAndAppendProxyServer("https", https_proxy.server, &spec);
-      EncodeAndAppendProxyServer("ftp", ftp_proxy.server, &spec);
-      EncodeAndAppendProxyServer("socks", socks_proxy.server, &spec);
+      ProxyConfigDictionary::EncodeAndAppendProxyServer(
+          url::kHttpScheme, http_proxy.server, &spec);
+      ProxyConfigDictionary::EncodeAndAppendProxyServer(
+          url::kHttpsScheme, https_proxy.server, &spec);
+      ProxyConfigDictionary::EncodeAndAppendProxyServer(
+          url::kFtpScheme, ftp_proxy.server, &spec);
+      ProxyConfigDictionary::EncodeAndAppendProxyServer(
+          kSocksScheme, socks_proxy.server, &spec);
       return ProxyConfigDictionary::CreateFixedServers(
           spec, bypass_rules.ToString());
     }
     default:
       break;
   }
   NOTREACHED() << "Unrecognized proxy config mode for preference";
   return NULL;
 }
 
 UIProxyConfig::ManualProxy* UIProxyConfig::MapSchemeToProxy(
     const std::string& scheme) {
-  if (scheme == "http")
+  if (scheme == url::kHttpScheme)
     return &http_proxy;
-  if (scheme == "https")
+  if (scheme == url::kHttpsScheme)
     return &https_proxy;
-  if (scheme == "ftp")
+  if (scheme == url::kFtpScheme)
     return &ftp_proxy;
-  if (scheme == "socks")
+  if (scheme == kSocksScheme)
     return &socks_proxy;
   NOTREACHED() << "Invalid scheme: " << scheme;
   return NULL;
 }
 
-// static
-void UIProxyConfig::EncodeAndAppendProxyServer(const std::string& url_scheme,
-                                               const net::ProxyServer& server,
-                                               std::string* spec) {
-  if (!server.is_valid())
-    return;
-
-  if (!spec->empty())
-    *spec += ';';
-
-  if (!url_scheme.empty()) {
-    *spec += url_scheme;
-    *spec += "=";
-  }
-  *spec += server.ToURI();
-}
-
 }  // namespace chromeos