Hidden windows should not keep Chrome alive.

chrome/browser/ui/apps/chrome_app_delegate.cc

Index: chrome/browser/ui/apps/chrome_app_delegate.cc
diff --git a/chrome/browser/ui/apps/chrome_app_delegate.cc b/chrome/browser/ui/apps/chrome_app_delegate.cc
index 36a96d221c074765a5b3e38ad80c71a43b74ebb0..3aa791ff7b6a5092240014a0d5715364fbaf2ade 100644
--- a/chrome/browser/ui/apps/chrome_app_delegate.cc
+++ b/chrome/browser/ui/apps/chrome_app_delegate.cc
@@ -304,6 +304,13 @@ void ChromeAppDelegate::SetTerminatingCallback(const base::Closure& callback) {
   terminating_callback_ = callback;
 }
 
+void ChromeAppDelegate::OnHide() {
+  keep_alive_.reset();

[benwells, 2015/01/28 02:42:10]

If an app window is created hidden does it get OnHide?

[jackhou1, 2015/02/03 06:05:13]

Yup, AppWindow::Init calls Hide() if the window is initially hidden.

+}
+void ChromeAppDelegate::OnShow() {

[benwells, 2015/01/28 02:42:10]

Nit: Blank line above.

[jackhou1, 2015/02/03 06:05:13]

Done.

+  keep_alive_.reset(new ScopedKeepAlive);
+}
+
 void ChromeAppDelegate::Observe(int type,
                                 const content::NotificationSource& source,
                                 const content::NotificationDetails& details) {