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

Unified Diff: chrome/browser/ui/apps/chrome_app_delegate.h

Issue 1708343002: Add ScopedKeepAlive to c/b/lifetime (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove KeepAliveOrigin::TEST Created 4 years, 10 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
« no previous file with comments | « chrome/browser/ui/app_list/profile_loader.cc ('k') | chrome/browser/ui/apps/chrome_app_delegate.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/apps/chrome_app_delegate.h
diff --git a/chrome/browser/ui/apps/chrome_app_delegate.h b/chrome/browser/ui/apps/chrome_app_delegate.h
index f1fed82e0271837e64108fa916653d261a37f1ec..37432322e4cfc2dbc249047835edbd02ac2a83dc 100644
--- a/chrome/browser/ui/apps/chrome_app_delegate.h
+++ b/chrome/browser/ui/apps/chrome_app_delegate.h
@@ -20,9 +20,9 @@ class ScopedKeepAlive;
class ChromeAppDelegate : public extensions::AppDelegate,
public content::NotificationObserver {
public:
- // Pass a ScopedKeepAlive to prevent the browser process from shutting down
- // while this object exists.
- explicit ChromeAppDelegate(scoped_ptr<ScopedKeepAlive> keep_alive);
+ // Params:
+ // keep_alive: Whether this object should keep the browser alive.
+ explicit ChromeAppDelegate(bool keep_alive);
~ChromeAppDelegate() override;
static void DisableExternalOpenForTesting();
« no previous file with comments | « chrome/browser/ui/app_list/profile_loader.cc ('k') | chrome/browser/ui/apps/chrome_app_delegate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698