Enable login screen apps.

chrome/browser/extensions/chrome_process_manager_delegate.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/extensions/chrome_process_manager_delegate.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "build/build_config.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_notification_types.h"
+#include "chrome/browser/extensions/extension_management.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/common/chrome_switches.h"
 #include "components/user_manager/user_manager.h"
 #include "content/public/browser/notification_service.h"
 #include "extensions/browser/extension_system.h"
 #include "extensions/browser/process_manager.h"
 #include "extensions/browser/process_manager_factory.h"
 #include "extensions/common/one_shot_event.h"
 
+#if defined(OS_CHROMEOS)
+#include "chrome/browser/chromeos/profiles/profile_helper.h"
+#endif
+
 namespace extensions {
 
 ChromeProcessManagerDelegate::ChromeProcessManagerDelegate() {
   registrar_.Add(this,
                  chrome::NOTIFICATION_BROWSER_WINDOW_READY,
                  content::NotificationService::AllSources());
   registrar_.Add(this,
                  chrome::NOTIFICATION_PROFILE_CREATED,
                  content::NotificationService::AllSources());
   registrar_.Add(this,
                  chrome::NOTIFICATION_PROFILE_DESTROYED,
                  content::NotificationService::AllSources());
 }
 
 ChromeProcessManagerDelegate::~ChromeProcessManagerDelegate() {
 }
 
 bool ChromeProcessManagerDelegate::IsBackgroundPageAllowed(
     content::BrowserContext* context) const {
   Profile* profile = static_cast<Profile*>(context);
 
   bool is_normal_session = !profile->IsGuestSession() &&
                            !profile->IsSystemProfile();
 #if defined(OS_CHROMEOS)
-  is_normal_session = is_normal_session &&
-                      user_manager::UserManager::Get()->IsUserLoggedIn();
+  if (is_normal_session) {
+    const bool user_logged_in =
+        user_manager::UserManager::Get()->IsUserLoggedIn();
+    const bool is_signin_profile =
+        chromeos::ProfileHelper::IsSigninProfile(profile);
+    const bool login_apps_enabled =
+        base::CommandLine::ForCurrentProcess()->HasSwitch(
+            switches::kEnableLoginApps);
+    const bool login_apps_installed =
+        !ExtensionManagementFactory::GetForBrowserContext(profile)

[emaxx, 2016/07/15 16:31:09]

nit: The function accetps BrowserContext*, so there is no reason to pass profile
here instead of context.

[achuithb, 2016/07/21 00:13:43]

Done.

+             ->GetForceInstallList()
+             ->empty();
+    is_normal_session =
+        user_logged_in ||
+        (is_signin_profile && login_apps_enabled && login_apps_installed);

[emaxx, 2016/07/15 16:31:09]

I think it's better to use another boolean variable for this login-related
stuff, as the name "is_normal_session" doesn't really correspond to the case of
the login screen.

[achuithb, 2016/07/21 00:13:43]

Done.

+  }
 #endif
 
   // Disallow if the current session is a Guest mode session or login screen but
   // the current browser context is *not* off-the-record. Such context is
   // artificial and background page shouldn't be created in it.
   // http://crbug.com/329498
   return is_normal_session || profile->IsOffTheRecord();
 }
 
 bool ChromeProcessManagerDelegate::DeferCreatingStartupBackgroundHosts(
@@ skipping 104 matching lines @@
     ProcessManager* incognito_manager =
         ProcessManagerFactory::GetForBrowserContextIfExists(
             profile->GetOffTheRecordProfile());
     if (incognito_manager) {
       incognito_manager->CloseBackgroundHosts();
     }
   }
 }
 
 }  // namespace extensions