Add ScopedKeepAlive to c/b/lifetime

chrome/browser/ui/views/panels/panel_view.cc

Index: chrome/browser/ui/views/panels/panel_view.cc
diff --git a/chrome/browser/ui/views/panels/panel_view.cc b/chrome/browser/ui/views/panels/panel_view.cc
index 9775a2de76037edc08dd156748942d37f79553cf..ba6ef05c71173d62392b45fa2d14c8249267f03f 100644
--- a/chrome/browser/ui/views/panels/panel_view.cc
+++ b/chrome/browser/ui/views/panels/panel_view.cc
@@ -14,13 +14,14 @@
 #include "base/strings/utf_string_conversions.h"
 #include "build/build_config.h"
 #include "chrome/app/chrome_command_ids.h"
+#include "chrome/browser/lifetime/keep_alive_types.h"
+#include "chrome/browser/lifetime/scoped_keep_alive.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/host_desktop.h"
 #include "chrome/browser/ui/panels/panel.h"
 #include "chrome/browser/ui/panels/panel_bounds_animation.h"
 #include "chrome/browser/ui/panels/panel_manager.h"
 #include "chrome/browser/ui/panels/stacked_panel_collection.h"
-#include "chrome/browser/ui/views/auto_keep_alive.h"
 #include "chrome/browser/ui/views/panels/panel_frame_view.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/render_widget_host_view.h"
@@ -300,8 +301,11 @@ PanelView::PanelView(Panel* panel, const gfx::Rect& bounds, bool always_on_top)
   window_->set_focus_on_creation(false);
   window_->AddObserver(this);
 
+#if !defined(OS_CHROMEOS)

[sky, 2016/02/25 18:43:50]

I think want !USE_ASH here.

[dgn, 2016/02/25 20:08:50]

Isn't there[1] where the count is incremented to balance the call being skipped
here?

It's guarded by OS_CHROMEOS

Another place it could have been done is [2], but it's guarded by !AURA so I'm
quite confused there...


[1]:
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...
[2]:
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...

[sky, 2016/02/25 23:27:49]

I think that increment should be outside the ifdef. It made sense at one time,
but not any more.
I'm not sure on that one. If that code is only compiled on
windows/linux/chromeos, then USE_ASH is what you want. If that code is compiled
on mac, then it should be USE_ASH or MAC.

[dgn, 2016/02/26 11:51:59]

Done.

[dgn, 2016/02/26 11:51:59]

Thanks, I'll keep that in mind when changing that file.
Okay. Changing that is out of scope if this CL anyway. I filed
https://crbug.com/590173 to track that. I'll just close it if it turns out to be
fine as it is.

   // Prevent the browser process from shutting down while this window is open.
-  keep_alive_.reset(new AutoKeepAlive(GetNativePanelWindow()));
+  // Chrome OS already has a mechanism to always stay alive and skips this.
+  keep_alive_.reset(new ScopedKeepAlive(KeepAliveOrigin::PANEL_VIEW));
+#endif  // !defined(OS_CHROMEOS)
 
   web_view_ = new views::WebView(NULL);
   AddChildView(web_view_);