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

Unified Diff: chrome/browser/ui/webui/chromeos/login/network_state_informer.cc

Issue 2836453003: tray: Show warning in network tray when VPN or proxy used. (Closed)
Patch Set: Created 3 years, 8 months 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/login/network_state_informer.cc
diff --git a/chrome/browser/ui/webui/chromeos/login/network_state_informer.cc b/chrome/browser/ui/webui/chromeos/login/network_state_informer.cc
index 3b3c24ea701a15e5d9df715612859699b21fa5be..66bf1f7b4f76e1b367957ed5993ae7e89cc0bb94 100644
--- a/chrome/browser/ui/webui/chromeos/login/network_state_informer.cc
+++ b/chrome/browser/ui/webui/chromeos/login/network_state_informer.cc
@@ -29,20 +29,6 @@ const char kNetworkStateCaptivePortal[] = "behind captive portal";
const char kNetworkStateConnecting[] = "connecting";
const char kNetworkStateProxyAuthRequired[] = "proxy auth required";
-bool HasDefaultNetworkProxyConfigured() {
- const NetworkState* network =
- NetworkHandler::Get()->network_state_handler()->DefaultNetwork();
- if (!network)
- return false;
- onc::ONCSource onc_source = onc::ONC_SOURCE_NONE;
- std::unique_ptr<ProxyConfigDictionary> proxy_dict =
- proxy_config::GetProxyConfigForNetwork(
- NULL, g_browser_process->local_state(), *network, &onc_source);
- ProxyPrefs::ProxyMode mode;
- return (proxy_dict && proxy_dict->GetMode(&mode) &&
- mode == ProxyPrefs::MODE_FIXED_SERVERS);
-}
-
NetworkStateInformer::State GetStateForDefaultNetwork() {
const NetworkState* network =
NetworkHandler::Get()->network_state_handler()->DefaultNetwork();
@@ -62,13 +48,13 @@ NetworkStateInformer::State GetStateForDefaultNetwork() {
// NetworkPortalDetector's state of current network is unknown.
if (status == NetworkPortalDetector::CAPTIVE_PORTAL_STATUS_ONLINE ||
(status == NetworkPortalDetector::CAPTIVE_PORTAL_STATUS_UNKNOWN &&
- !HasDefaultNetworkProxyConfigured() &&
+ !NetworkStateInformer::HasDefaultNetworkProxyConfigured() &&
network->connection_state() == shill::kStateOnline)) {
return NetworkStateInformer::ONLINE;
}
if (status ==
NetworkPortalDetector::CAPTIVE_PORTAL_STATUS_PROXY_AUTH_REQUIRED &&
- HasDefaultNetworkProxyConfigured()) {
+ NetworkStateInformer::HasDefaultNetworkProxyConfigured()) {
return NetworkStateInformer::PROXY_AUTH_REQUIRED;
}
if (status == NetworkPortalDetector::CAPTIVE_PORTAL_STATUS_PORTAL ||
@@ -99,6 +85,21 @@ NetworkStateInformer::State GetStateForDefaultNetwork() {
} // namespace
+// static
+bool NetworkStateInformer::HasDefaultNetworkProxyConfigured() {
+ const NetworkState* network =
+ NetworkHandler::Get()->network_state_handler()->DefaultNetwork();
+ if (!network)
+ return false;
+ onc::ONCSource onc_source = onc::ONC_SOURCE_NONE;
+ std::unique_ptr<ProxyConfigDictionary> proxy_dict =
+ proxy_config::GetProxyConfigForNetwork(
+ NULL, g_browser_process->local_state(), *network, &onc_source);
+ ProxyPrefs::ProxyMode mode;
+ return (proxy_dict && proxy_dict->GetMode(&mode) &&
+ mode == ProxyPrefs::MODE_FIXED_SERVERS);
+}
+
NetworkStateInformer::NetworkStateInformer()
: state_(OFFLINE),
weak_ptr_factory_(this) {

Powered by Google App Engine
This is Rietveld 408576698