Enable login screen apps.

extensions/browser/process_manager_delegate.h

Index: extensions/browser/process_manager_delegate.h
diff --git a/extensions/browser/process_manager_delegate.h b/extensions/browser/process_manager_delegate.h
index 7cc48c0af74ee81ab9db0e1eddc88bcbe1580e10..777df6702d5c9d8c59c001980945ef7da81c3009 100644
--- a/extensions/browser/process_manager_delegate.h
+++ b/extensions/browser/process_manager_delegate.h
@@ -11,6 +11,8 @@ class BrowserContext;
 
 namespace extensions {
 
+class Extension;
+
 // Customization of ProcessManager for the extension system embedder.
 class ProcessManagerDelegate {
  public:
@@ -18,8 +20,8 @@ class 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