Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6206)

Unified Diff: chrome/browser/ui/webui/chromeos/proxy_settings_ui.cc

Issue 8102019: redesign and reimplement proxy config service and tracker, revise proxy ui on cros (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/chromeos/proxy_settings_ui.cc
===================================================================
--- chrome/browser/ui/webui/chromeos/proxy_settings_ui.cc (revision 108608)
+++ chrome/browser/ui/webui/chromeos/proxy_settings_ui.cc (working copy)
@@ -66,7 +66,7 @@
ProxySettingsUI::ProxySettingsUI(TabContents* contents)
: ChromeWebUI(contents),
proxy_settings_(NULL),
- proxy_handler_(new ProxyHandler) {
+ proxy_handler_(new ProxyHandler(GetProfile())) {
// |localized_strings| will be owned by ProxySettingsHTMLSource.
DictionaryValue* localized_strings = new DictionaryValue();
@@ -103,9 +103,10 @@
}
if (proxy_settings()) {
proxy_settings()->MakeActiveNetworkCurrent();
- std::string network = proxy_settings()->GetCurrentNetworkName();
- if (!network.empty())
- proxy_handler_->SetNetworkName(network);
+ std::string network_name;
+ GetProfile()->GetProxyConfigTracker()->UIGetCurrentNetworkName(
+ &network_name);
+ proxy_handler_->SetNetworkName(network_name);
}
}

Powered by Google App Engine
This is Rietveld 408576698