cros: Do not initialize gaia portal detector if not loading gaia

chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.cc

Index: chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.cc
diff --git a/chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.cc b/chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.cc
index 67f3955d0cff065655c886d05f624affa67a4076..6c52d63f3a81f565ab959b8444916103c2b6b552 100644
--- a/chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.cc
+++ b/chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.cc
@@ -243,7 +243,8 @@ GaiaScreenHandler::~GaiaScreenHandler() {
 void GaiaScreenHandler::MaybePreloadAuthExtension() {
   VLOG(1) << "MaybePreloadAuthExtension";
 
-  if (!network_portal_detector_) {
+  if (!network_portal_detector_ &&
+      !user_manager::UserManager::Get()->IsUserLoggedIn()) {

[xiyuan, 2017/06/06 21:54:29]

Maybe move ShouldLoadGaia() before this and bail out early if it returns false ?

[jdufault, 2017/06/06 21:58:37]

Add a comment explaining why we need this special case.

[Qiang(Joe) Xu, 2017/06/06 22:51:09]

Done.

[Qiang(Joe) Xu, 2017/06/06 22:51:09]

Done.

     NetworkPortalDetectorImpl* detector = new NetworkPortalDetectorImpl(
         g_browser_process->system_request_context(), false);
     detector->set_portal_test_url(GURL(kRestrictiveProxyURL));