Remove Increment/DecrementKeepAliveCount from AppsClient

chrome/browser/apps/app_browsertest_util.cc

Index: chrome/browser/apps/app_browsertest_util.cc
diff --git a/chrome/browser/apps/app_browsertest_util.cc b/chrome/browser/apps/app_browsertest_util.cc
index fe9540528181593ac23de7d272e09fd9d7ad4f39..65dacecf210969462a0c8b4c315e4156ce643aab 100644
--- a/chrome/browser/apps/app_browsertest_util.cc
+++ b/chrome/browser/apps/app_browsertest_util.cc
@@ -6,6 +6,7 @@
 
 #include "base/command_line.h"
 #include "base/strings/stringprintf.h"
+#include "chrome/browser/apps/scoped_keep_alive.h"
 #include "chrome/browser/extensions/api/tabs/tabs_api.h"
 #include "chrome/browser/extensions/extension_function_test_utils.h"
 #include "chrome/browser/ui/apps/chrome_app_delegate.h"
@@ -201,7 +202,9 @@ AppWindow* PlatformAppBrowserTest::CreateAppWindowFromParams(
     const Extension* extension,
     const AppWindow::CreateParams& params) {
   AppWindow* window =
-      new AppWindow(browser()->profile(), new ChromeAppDelegate(), extension);
+      new AppWindow(browser()->profile(),
+                    new ChromeAppDelegate(scoped_ptr<ScopedKeepAlive>()),

[tapted, 2014/09/24 07:12:41]

to maintain behaviour in tests, I think this should pass a new ScopedKeepAlive.
I think they typically have a browser window open the whole time, which holds
the keepalive. However, there will be more realistic coverage with the keepalive
on the app window as well.

[hashimoto, 2014/09/24 09:03:46]

Done.

+                    extension);
   window->Init(GURL(std::string()), new AppWindowContentsImpl(window), params);
   return window;
 }