Enable login screen apps.

extensions/browser/process_manager.cc

 // Copyright 2013 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 "extensions/browser/process_manager.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/location.h"
@@ skipping 88 matching lines @@
                             const GURL& url) override;
   scoped_refptr<content::SiteInstance> GetSiteInstanceForURL(const GURL& url)
       override;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(IncognitoProcessManager);
 };
 
 static void CreateBackgroundHostForExtensionLoad(
     ProcessManager* manager, const Extension* extension) {
-  DVLOG(1) << "CreateBackgroundHostForExtensionLoad " << extension->id();
   if (BackgroundInfo::HasPersistentBackgroundPage(extension))
     manager->CreateBackgroundHost(extension,
                                   BackgroundInfo::GetBackgroundURL(extension));
 }
 
 void PropagateExtensionWakeResult(const base::Callback<void(bool)>& callback,
                                   extensions::ExtensionHost* host) {
   callback.Run(host != nullptr);
 }
 
@@ skipping 238 matching lines @@
 bool ProcessManager::CreateBackgroundHost(const Extension* extension,
                                           const GURL& url) {
   // Hosted apps are taken care of from BackgroundContentsService. Ignore them
   // here.
   if (extension->is_hosted_app())
     return false;
 
   // Don't create hosts if the embedder doesn't allow it.
   ProcessManagerDelegate* delegate =
       ExtensionsBrowserClient::Get()->GetProcessManagerDelegate();
-  if (delegate && !delegate->IsBackgroundPageAllowed(browser_context_))
+  if (delegate &&
+      !delegate->IsExtensionBackgroundPageAllowed(browser_context_, extension))
     return false;
 
   // Don't create multiple background hosts for an extension.
   if (GetBackgroundHostForExtension(extension->id()))
     return true;  // TODO(kalman): return false here? It might break things...
 
+  DVLOG(1) << "CreateBackgroundHost " << extension->id();

[achuithb, 2017/03/22 00:15:53]

This comment is preferable here rather than line 109 because when we see this
line printed, we're guaranteed that an extension host has been created.

   ExtensionHost* host =
       new ExtensionHost(extension, GetSiteInstanceForURL(url).get(), url,
                         VIEW_TYPE_EXTENSION_BACKGROUND_PAGE);
   host->CreateRenderViewSoon();
   OnBackgroundHostCreated(host);
   return true;
 }
 
 void ProcessManager::MaybeCreateStartupBackgroundHosts() {
   if (startup_background_hosts_created_)
     return;
 
   // The embedder might disallow background pages entirely.
   ProcessManagerDelegate* delegate =
       ExtensionsBrowserClient::Get()->GetProcessManagerDelegate();
-  if (delegate && !delegate->IsBackgroundPageAllowed(browser_context_))
+  if (delegate &&
+      !delegate->AreBackgroundPagesAllowedForContext(browser_context_))
     return;
 
   // The embedder might want to defer background page loading. For example,
   // Chrome defers background page loading when it is launched to show the app
   // list, then triggers a load later when a browser window opens.
   if (delegate &&
       delegate->DeferCreatingStartupBackgroundHosts(browser_context_))
     return;
 
   CreateStartupBackgroundHosts();
@@ skipping 589 matching lines @@
   if (extension && !IncognitoInfo::IsSplitMode(extension)) {
     BrowserContext* original_context =
         ExtensionsBrowserClient::Get()->GetOriginalContext(browser_context());
     return ProcessManager::Get(original_context)->GetSiteInstanceForURL(url);
   }
 
   return ProcessManager::GetSiteInstanceForURL(url);
 }
 
 }  // namespace extensions