ChromeOS: Implement CaptivePortalAuthenticationIgnoresProxy policy.

chrome/browser/chromeos/net/network_portal_notification_controller.cc

 // Copyright 2014 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/chromeos/net/network_portal_notification_controller.h"
 
 #include "ash/shell.h"
 #include "ash/system/system_notifier.h"
 #include "ash/system/tray/system_tray_notifier.h"
 #include "base/basictypes.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_loop.h"
 #include "base/metrics/histogram.h"
+#include "base/prefs/pref_service.h"
 #include "base/strings/string16.h"
 #include "base/strings/utf_string_conversions.h"
+#include "chrome/browser/browser_process.h"
+#include "chrome/browser/browser_process_platform_part.h"
 #include "chrome/browser/chromeos/mobile/mobile_activator.h"
 #include "chrome/browser/chromeos/net/network_portal_web_dialog.h"
+#include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h"
+#include "chrome/browser/chromeos/policy/consumer_management_service.h"
 #include "chrome/browser/chromeos/profiles/profile_helper.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/browser_dialogs.h"
 #include "chrome/browser/ui/scoped_tabbed_browser_displayer.h"
 #include "chrome/browser/ui/singleton_tabs.h"
+#include "chrome/common/pref_names.h"
 #include "chrome/grit/generated_resources.h"
 #include "chrome/grit/theme_resources.h"
 #include "chromeos/chromeos_switches.h"
 #include "chromeos/network/network_state.h"
 #include "components/captive_portal/captive_portal_detector.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/message_center/message_center.h"
 #include "ui/message_center/notification.h"
 #include "ui/message_center/notification_types.h"
@@ skipping 59 matching lines @@
       NetworkPortalNotificationController::USER_ACTION_METRIC_COUNT);
 }
 
 void NetworkPortalNotificationControllerDelegate::Click() {
   clicked_ = true;
   UMA_HISTOGRAM_ENUMERATION(
       NetworkPortalNotificationController::kUserActionMetric,
       NetworkPortalNotificationController::USER_ACTION_METRIC_CLICKED,
       NetworkPortalNotificationController::USER_ACTION_METRIC_COUNT);
 
-  if (CommandLine::ForCurrentProcess()->HasSwitch(
-          chromeos::switches::kEnableCaptivePortalBypassProxy)) {
+  Profile* profile = ProfileManager::GetActiveUserProfile();
+
+  const bool bypass_proxy_switch_present =
+      CommandLine::ForCurrentProcess()->HasSwitch(
+          chromeos::switches::kEnableCaptivePortalBypassProxy);
+  const bool use_incognito_profile =
+      bypass_proxy_switch_present
+          ? (profile &&
+             profile->GetPrefs()->GetBoolean(

[bartfab (slow), 2015/01/07 17:38:42]

Nit: #include "base/prefs/pref_service.h"

[Alexander Alekseev, 2015/01/12 18:41:09]

Done.

+                 prefs::kCaptivePortalAuthenticationIgnoresProxy))

[bartfab (slow), 2015/01/07 17:38:42]

Nit: #include "chrome/common/pref_names.h"

[Alexander Alekseev, 2015/01/12 18:41:09]

Done.

+          : false;
+
+  if (use_incognito_profile) {
     if (controller_)
       controller_->ShowDialog();
   } else {
-    Profile* profile = ProfileManager::GetActiveUserProfile();
     if (!profile)
       return;
     chrome::ScopedTabbedBrowserDisplayer displayer(
         profile, chrome::HOST_DESKTOP_TYPE_ASH);
     GURL url(captive_portal::CaptivePortalDetector::kDefaultURL);
     chrome::ShowSingletonTab(displayer.browser(), url);
   }
   CloseNotification();
 }
 
 }  // namespace
 
 // static
 const char NetworkPortalNotificationController::kNotificationId[] =
     "chrome://net/network_portal_detector";
 
 // static
 const char NetworkPortalNotificationController::kNotificationMetric[] =
     "CaptivePortal.Notification.Status";
 
 // static
 const char NetworkPortalNotificationController::kUserActionMetric[] =
     "CaptivePortal.Notification.UserAction";
 
 NetworkPortalNotificationController::NetworkPortalNotificationController()
-    : dialog_(nullptr) {
+    : dialog_(nullptr), ignore_no_network_for_testing_(false) {
 }
 
 NetworkPortalNotificationController::~NetworkPortalNotificationController() {}
 
 void NetworkPortalNotificationController::OnPortalDetectionCompleted(
     const NetworkState* network,
     const NetworkPortalDetector::CaptivePortalState& state) {
   if (!IsPortalNotificationEnabled())
     return;
 
   if (!network ||
       state.status != NetworkPortalDetector::CAPTIVE_PORTAL_STATUS_PORTAL) {
     last_network_path_.clear();
 
-    if (dialog_)
+    // Do not close dialog in browser tests on "no network" errors.
+    if (dialog_ &&
+        (!ignore_no_network_for_testing_ ||

[bartfab (slow), 2015/01/07 17:38:42]

This conditional makes |ignore_no_network_for_testing_| prevent the dialog from
closing in *all* cases, not just for a specific class of "no network" errors.

[Alexander Alekseev, 2015/01/12 18:41:09]

This is intentional.

[bartfab (slow), 2015/01/14 11:00:07]

It does not seem to match the comment though.

[Alexander Alekseev, 2015/01/16 22:51:48]

Done.

(The comment was really outdated, sorry.)

+         state.status == NetworkPortalDetector::CAPTIVE_PORTAL_STATUS_ONLINE)) {

[bartfab (slow), 2015/01/07 17:38:42]

If |ignore_no_network_for_testing_| is false (the default), the dialog used to
be always closed. Now, it will not be closed if |state.status ==
NetworkPortalDetector::CAPTIVE_PORTAL_STATUS_ONLINE|. Is this intentional?

[Alexander Alekseev, 2015/01/12 18:41:09]

Yes.
In browser_tests we have very strange condition: network portal is forced to
show, but network state is not changed, so it is usually ONLINE. And in a few
moments after network portal is shown, we receive event "network is online"
which forces portal dialog to close before we check that it exists.

       dialog_->Close();
+    }
 
     CloseNotification();
     return;
   }
 
   // Don't do anything if we're currently activating the device.
   if (MobileActivator::GetInstance()->RunningActivation())
     return;
 
   // Don't do anything if notification for |network| already was
@@ skipping 38 matching lines @@
 }
 
 void NetworkPortalNotificationController::OnDialogDestroyed(
     const NetworkPortalWebDialog* dialog) {
   if (dialog == dialog_) {
     dialog_ = nullptr;
     ProfileHelper::Get()->ClearSigninProfile(base::Closure());
   }
 }
 
+void NetworkPortalNotificationController::SetIgnoreNoNetworkForTesting() {
+  ignore_no_network_for_testing_ = true;
+}
+
+const NetworkPortalWebDialog*
+NetworkPortalNotificationController::GetDialogForTesting() const {
+  return dialog_;
+}
+
 }  // namespace chromeos