Add ScopedKeepAlive to c/b/lifetime

chrome/browser/ui/app_list/app_list_service_views.cc

Index: chrome/browser/ui/app_list/app_list_service_views.cc
diff --git a/chrome/browser/ui/app_list/app_list_service_views.cc b/chrome/browser/ui/app_list/app_list_service_views.cc
index 8ae98e5e8010d4f451a95b732ffdad555759f65e..53a5d3f1c6fc2836837c46de4acec48b236f2c22 100644
--- a/chrome/browser/ui/app_list/app_list_service_views.cc
+++ b/chrome/browser/ui/app_list/app_list_service_views.cc
@@ -6,13 +6,20 @@
 
 #include <utility>
 
-#include "chrome/browser/apps/scoped_keep_alive.h"
+#include "chrome/browser/lifetime/keep_alive_options.h"
+#include "chrome/browser/lifetime/scoped_keep_alive.h"
 #include "chrome/browser/ui/app_list/app_list_controller_delegate.h"
 #include "ui/app_list/app_list_switches.h"
 #include "ui/app_list/views/app_list_main_view.h"
 #include "ui/app_list/views/app_list_view.h"
 #include "ui/app_list/views/contents_view.h"
 
+namespace {
+
+static const KeepAliveOptions g_keep_alive_options = {"AppListServiceViews"};

[Bernhard Bauer, 2016/02/22 18:19:38]

Same as above. Also (as mentioned above), the anonymous namespace isn't really
necessary if you only have consts, so you could remove it (but OTOH, it doesn't
hurt either. I'm mostly mentioning this for completeness / education).

[dgn, 2016/02/22 19:07:19]

Thanks. Removed since there's nothing else in the anonymous namespace.

+
+}  // namespace
+
 AppListServiceViews::AppListServiceViews(
     scoped_ptr<AppListControllerDelegate> controller_delegate)
     : shower_(this),
@@ -116,7 +123,7 @@ void AppListServiceViews::ShowForProfileInternal(
     app_list::AppListModel::State state) {
   DCHECK(profile);
 
-  ScopedKeepAlive keep_alive;
+  ScopedKeepAlive keep_alive(&g_keep_alive_options);
 
   CreateForProfile(profile);