Migrate ProxyConfigServiceImpl to NetworkStateHandler and NetworkProfileHandler.

chrome/browser/ui/webui/chromeos/login/network_state_informer.cc

 // Copyright (c) 2012 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/ui/webui/chromeos/login/network_state_informer.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "chrome/browser/chromeos/cros/cros_library.h"
-#include "chrome/browser/chromeos/proxy_config_service_impl.h"
+#include "chrome/browser/prefs/proxy_config_dictionary.h"
+#include "chrome/browser/prefs/proxy_prefs.h"
 #include "chrome/common/chrome_notification_types.h"
+#include "chromeos/network/network_state.h"
+#include "chromeos/network/network_state_handler.h"
 #include "net/proxy/proxy_config.h"
 
 namespace {
 
 // Timeout to smooth temporary network state transitions for flaky networks.
 const int kNetworkStateCheckDelaySec = 3;
 
 }  // namespace
 
 namespace chromeos {
@@ skipping 177 matching lines @@
     if (network->online())
       return ONLINE;
     if (network->restricted_pool())
       return CAPTIVE_PORTAL;
   }
   return OFFLINE;
 }
 
 bool NetworkStateInformer::IsProxyConfigured(const Network* network) {
   DCHECK(network);
-  ProxyStateMap::iterator it = proxy_state_map_.find(network->unique_id());
-  if (it != proxy_state_map_.end() &&
-      it->second.proxy_config == network->proxy_config()) {
-    return it->second.configured;
-  }
-  net::ProxyConfig proxy_config;
-  if (!ProxyConfigServiceImpl::ParseProxyConfig(network, &proxy_config))
-    return false;
-  bool configured = !proxy_config.proxy_rules().empty();
-  proxy_state_map_[network->unique_id()] =
-      ProxyState(network->proxy_config(), configured);
-  return configured;
+  const NetworkState* network_state =
+      NetworkStateHandler::Get()->GetNetworkState(network->service_path());
+  DCHECK(network_state);

[stevenjb, 2013/05/15 17:44:14]

nit: Highly unlikely but not impossible, e.g. if this happens as a side effect
of NetworkLibrary receiving a Manger update before NetworkStateHandler processes
the same update. I'd rather handle this with a warning (i.e. return false) than
crash.

[pneubeck (no reviews), 2013/05/15 20:24:48]

This is still waiting on Gaurav's commit. I should all be NetworkStateHandler.

+
+  ProxyConfigDictionary proxy_dict(&network_state->proxy_config());
+  ProxyPrefs::ProxyMode mode;
+  return !proxy_dict.GetMode(&mode) || mode == ProxyPrefs::MODE_FIXED_SERVERS;
 }
 
 }  // namespace chromeos