Allow closing root window, or pressing the power button to shutdown ash_shell, and chrome /w chrome…

ash/wm/power_button_controller.cc

 // Copyright (c) 2012 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 "ash/wm/power_button_controller.h"
 
 #include "ash/ash_switches.h"
 #include "ash/shell.h"
+#include "ash/shell_delegate.h"
 #include "ash/shell_window_ids.h"
 #include "ash/wm/cursor_manager.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/time.h"
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/aura/env.h"
 #include "ui/aura/root_window.h"
 #include "ui/aura/shared/compound_event_filter.h"
 #include "ui/aura/window.h"
@@ skipping 446 matching lines @@
   if (!shutting_down_)
     StartShutdownAnimationAndRequestShutdown();
 }
 
 void PowerButtonController::OnRootWindowResized(const aura::RootWindow* root,
                                                 const gfx::Size& new_size) {
   if (background_layer_.get())
     background_layer_->SetBounds(gfx::Rect(root->bounds().size()));
 }
 
+void PowerButtonController::OnRootWindowHostClosed(const aura::RootWindow*) {
+  if(Shell::GetInstance() && Shell::GetInstance()->delegate()) {
+    Shell::GetInstance()->delegate()->Exit();

[sadrul, 2012/08/07 13:29:03]

Is this change to make sure chrome terminates if the host window is closed by
the user? Can this cause problems if the host-window is closing because chrome
is already terminating?

[Harry McCleave, 2012/08/07 18:30:14]

In its current form the host window doesn't actually close, the event that
triggers this is the user clicking the 'x' to close the host window but this
event is ignored, this change causes the application to exit at that point,
though perhaps isn't the correct way to do so?

[sadrul, 2012/08/07 22:13:24]

If the user terminates chrome from within chrome (e.g. pressing ctrl-shift-q),
will this code trigger? If it does, will this cause problems?

+  }
+}
+
 bool PowerButtonController::LoggedInAsNonGuest() const {
   if (login_status_ == user::LOGGED_IN_NONE)
     return false;
   if (login_status_ == user::LOGGED_IN_GUEST)
     return false;
   // TODO(mukai): think about kiosk mode.
   return true;
 }
 
 void PowerButtonController::OnLockTimeout() {
@@ skipping 88 matching lines @@
     root_layer->StackAtBottom(background_layer_.get());
   }
   background_layer_->SetVisible(true);
 }
 
 void PowerButtonController::HideBackgroundLayer() {
   background_layer_.reset();
 }
 
 }  // namespace ash