Refactor proxy handling for ChromeOS to not go through the CrosSettings interface.

chrome/browser/ui/webui/chromeos/proxy_settings_ui.cc

 // Copyright (c) 2011 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/proxy_settings_ui.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop.h"
 #include "base/values.h"
 #include "chrome/browser/chromeos/cros_settings.h"
+#include "chrome/browser/chromeos/proxy_config_service_impl.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/webui/chrome_url_data_manager.h"
 #include "chrome/browser/ui/webui/options/chromeos/core_chromeos_options_handler.h"
 #include "chrome/browser/ui/webui/options/chromeos/proxy_handler.h"
+#include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/jstemplate_builder.h"
 #include "chrome/common/url_constants.h"
 #include "content/browser/tab_contents/tab_contents.h"
+#include "content/public/browser/notification_service.h"
 #include "grit/browser_resources.h"
 #include "ui/base/resource/resource_bundle.h"
 
 namespace {
 
 class ProxySettingsHTMLSource : public ChromeURLDataManager::DataSource {
  public:
   explicit ProxySettingsHTMLSource(DictionaryValue* localized_strings);
 
   // Called when the network layer has requested a resource underneath
@@ skipping 30 matching lines @@
 
   SendResponse(request_id, base::RefCountedString::TakeString(&full_html));
 }
 
 }  // namespace
 
 namespace chromeos {
 
 ProxySettingsUI::ProxySettingsUI(TabContents* contents)
     : ChromeWebUI(contents),
-      proxy_settings_(NULL),
-      proxy_handler_(new ProxyHandler(GetProfile())) {
+      proxy_handler_(new ProxyHandler()) {
   // |localized_strings| will be owned by ProxySettingsHTMLSource.
   DictionaryValue* localized_strings = new DictionaryValue();
 
   CoreChromeOSOptionsHandler* core_handler = new CoreChromeOSOptionsHandler();
   core_handler->set_handlers_host(this);
   core_handler->GetLocalizedValues(localized_strings);
   AddMessageHandler(core_handler->Attach(this));
 
   proxy_handler_->GetLocalizedValues(localized_strings);
   AddMessageHandler(proxy_handler_->Attach(this));
@@ skipping 14 matching lines @@
   }
   proxy_handler_ = NULL;  // Weak ptr that is owned by base class, nullify it.
 }
 
 void ProxySettingsUI::InitializeHandlers() {
   std::vector<WebUIMessageHandler*>::iterator iter;
   // Skip over the generic handler.
   for (iter = handlers_.begin() + 1; iter != handlers_.end(); ++iter) {
     (static_cast<OptionsPageUIHandler*>(*iter))->Initialize();
   }
-  if (proxy_settings()) {
-    proxy_settings()->MakeActiveNetworkCurrent();
-    std::string network_name;
-    GetProfile()->GetProxyConfigTracker()->UIGetCurrentNetworkName(
-        &network_name);
-    proxy_handler_->SetNetworkName(network_name);
-  }
-}
-
-chromeos::ProxyCrosSettingsProvider* ProxySettingsUI::proxy_settings() {
-  if (!proxy_settings_) {
-    proxy_settings_ = static_cast<chromeos::ProxyCrosSettingsProvider*>(
-      chromeos::CrosSettings::Get()->GetProvider("cros.session.proxy"));
-    if (!proxy_settings_)
-      NOTREACHED() << "Error getting access to proxy cros settings provider";
-  }
-  return proxy_settings_;
+  GetProfile()->GetProxyConfigTracker()->UIMakeActiveNetworkCurrent();
+  std::string network_name;
+  GetProfile()->GetProxyConfigTracker()->UIGetCurrentNetworkName(
+      &network_name);
+  content::NotificationService::current()->Notify(
+      chrome::NOTIFICATION_CURRENT_NETWORK_CHANGED,
+      content::NotificationService::AllSources(),
+      content::NotificationService::NoDetails());
+  proxy_handler_->SetNetworkName(network_name);
 }
 
 }  // namespace chromeos