Enable login screen apps.

extensions/browser/process_manager_delegate.h

 // 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.
 
 #ifndef EXTENSIONS_BROWSER_PROCESS_MANAGER_DELEGATE_H_
 #define EXTENSIONS_BROWSER_PROCESS_MANAGER_DELEGATE_H_
 
 namespace content {
 class BrowserContext;
 };
 
 namespace extensions {
 
+class Extension;
+
 // Customization of ProcessManager for the extension system embedder.
 class ProcessManagerDelegate {
  public:
   virtual ~ProcessManagerDelegate() {}
 
   // Returns true if the embedder allows background pages for the given
   // |context|.

[emaxx, 2017/03/08 17:11:25]

nit: Please update the comment to talk about |extension| too. (Including that it
can be null and what the function returns in that case.)

[Devlin, 2017/03/08 17:24:32]

Can it be null?

[achuithb, 2017/03/09 14:31:06]

Done.

[achuithb, 2017/03/09 14:31:06]

Yes, there's 2 places where IsBackgroundPageAllowed was called (now 3), and
there's no extension available in one of them.

-  virtual bool IsBackgroundPageAllowed(
-      content::BrowserContext* context) const = 0;
+  virtual bool IsBackgroundPageAllowed(content::BrowserContext* context,
+                                       const Extension* extension) const = 0;
 
   // Returns true if the embedder wishes to defer starting up the renderers for
   // extension background pages. If the embedder returns true it must call
   // ProcessManager::MaybeCreateStartupBackgroundHosts() when it is ready. See
   // ChromeProcessManagerDelegate for examples of how this is useful.
   virtual bool DeferCreatingStartupBackgroundHosts(
       content::BrowserContext* context) const = 0;
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_BROWSER_PROCESS_MANAGER_DELEGATE_H_