Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(29)

Unified Diff: extensions/browser/process_manager_delegate.h

Issue 2149953002: Enable login screen apps. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Devlin and Maksim feedback Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..7a8d50d8788fd68e333c4fdd85cfda7f4d276f55 100644
--- a/extensions/browser/process_manager_delegate.h
+++ b/extensions/browser/process_manager_delegate.h
@@ -11,15 +11,17 @@ 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|.
- virtual bool IsBackgroundPageAllowed(
- content::BrowserContext* context) const = 0;
+ // |context|, and a given |extension|, which can optionally be nullptr.
+ virtual bool IsBackgroundPageAllowed(content::BrowserContext* context,
Devlin 2017/03/17 16:37:23 Let's break this into two functions: bool AreBackg
achuithb 2017/03/22 00:15:53 Done.
+ 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

Powered by Google App Engine
This is Rietveld 408576698