Decouple EventClientImpl and root window. Check containers on the same root window as focused windo…

ash/wm/power_button_controller.cc

Index: ash/wm/power_button_controller.cc
diff --git a/ash/wm/power_button_controller.cc b/ash/wm/power_button_controller.cc
index bd9bb911ac2c52702eb5dd1e700912104aebae51..4f05f1b14d2f4c440145e6d2c8853e2e0ff055a5 100644
--- a/ash/wm/power_button_controller.cc
+++ b/ash/wm/power_button_controller.cc
@@ -13,6 +13,10 @@
 #include "ui/aura/root_window.h"
 #include "ui/aura/shared/compound_event_filter.h"
 
+#if defined(OS_CHROMEOS)
+#include "base/chromeos/chromeos_version.h"
+#endif
+
 namespace ash {
 
 namespace {
@@ -307,6 +311,13 @@ void PowerButtonController::OnShutdownTimeout() {
 
 void PowerButtonController::OnRealShutdownTimeout() {
   DCHECK(shutting_down_);
+#if defined(OS_CHROMEOS)

[Daniel Erat, 2012/10/08 14:41:51]

this code seems a bit strange.  you have an OS_CHROMEOS ifdef, but i think that
this class is completely unsafe to use on non-chrome-os builds, since i believe
that delegate_ will be null there (at least, the only
PowerButtonControllerDelegate implementation that i see is
PowerButtonControllerDelegateChromeos).

a better long-term solution might be a stub PowerButtonControllerDelegate that
we use on windows and chrome os builds when not running on devices that just
exits in response to RequestShutdown() calls.

(doesn't need to happen in this change, though.)

[oshima, 2012/10/08 17:52:59]

Chances are this class may not make sense at all on windows (is it possible to
intercept power button on windows?), although I'm not 100% sure. We may still
want
to keep it for debugging though.

[Daniel Erat, 2012/10/08 18:24:25]

Yeah, I was just suggesting this for debugging.

+  if (!base::chromeos::IsRunningOnChromeOS()) {
+    ShellDelegate* delegate = Shell::GetInstance()->delegate();
+    if (delegate)

[Daniel Erat, 2012/10/08 14:41:51]

random question: under what circumstances do we not create a ShellDelegate?

[oshima, 2012/10/08 17:52:59]

There are a few unittests that create Shell with NULL delegates. 
It's unlikely for these tests to hit this code, but other places in ash
assumes this can be NULL, so I'd like to do the same.

+      delegate->Exit();
+  }
+#endif
   delegate_->RequestShutdown();
 }