Add NetworkHandler to own network handlers in src/chromeos/network

chrome/browser/chromeos/policy/network_configuration_updater_impl.cc

Index: chrome/browser/chromeos/policy/network_configuration_updater_impl.cc
diff --git a/chrome/browser/chromeos/policy/network_configuration_updater_impl.cc b/chrome/browser/chromeos/policy/network_configuration_updater_impl.cc
index bf7c7adae6fe245f88034f5f64daf2ec37ad4353..30b15ba4439ceee8ac5a258960beeff2bb377393 100644
--- a/chrome/browser/chromeos/policy/network_configuration_updater_impl.cc
+++ b/chrome/browser/chromeos/policy/network_configuration_updater_impl.cc
@@ -21,12 +21,10 @@ namespace policy {
 
 NetworkConfigurationUpdaterImpl::NetworkConfigurationUpdaterImpl(
     PolicyService* policy_service,
-    chromeos::ManagedNetworkConfigurationHandler* network_config_handler,

[pneubeck (no reviews), 2013/05/13 09:17:06]

What is the improvement here? How will it make writing the missing test easier
(which will use a StrictMock of the config handler)?

[stevenjb, 2013/05/13 23:49:35]

See meta comment

     scoped_ptr<chromeos::CertificateHandler> certificate_handler)
     : policy_change_registrar_(
           policy_service, PolicyNamespace(POLICY_DOMAIN_CHROME, std::string())),
       policy_service_(policy_service),
-      network_config_handler_(network_config_handler),
       certificate_handler_(certificate_handler.Pass()) {
   policy_change_registrar_.Observe(
       key::kDeviceOpenNetworkConfiguration,
@@ -96,7 +94,8 @@ void NetworkConfigurationUpdaterImpl::ApplyNetworkConfiguration(
 
   std::string userhash = onc_source == chromeos::onc::ONC_SOURCE_USER_POLICY ?
       hashed_username_ : std::string();
-  network_config_handler_->SetPolicy(onc_source, userhash, network_configs);
+  chromeos::NetworkHandler::Get()->managed_network_configuration_handler()->
+      SetPolicy(onc_source, userhash, network_configs);
 
   scoped_ptr<net::CertificateList> web_trust_certs(new net::CertificateList);
   certificate_handler_->ImportCertificates(