Hidden windows should not keep Chrome alive.

chrome/browser/ui/apps/chrome_app_delegate.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/apps/chrome_app_delegate.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/app_mode/app_mode_utils.h"
 #include "chrome/browser/apps/scoped_keep_alive.h"
@@ skipping 286 matching lines @@
 
 bool ChromeAppDelegate::IsWebContentsVisible(
     content::WebContents* web_contents) {
   return platform_util::IsVisible(web_contents->GetNativeView());
 }
 
 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) {
   switch (type) {
     case chrome::NOTIFICATION_APP_TERMINATING:
       if (!terminating_callback_.is_null())
         terminating_callback_.Run();
       break;
     default:
       NOTREACHED() << "Received unexpected notification";
   }
 }