Converts WmActivationObserver to aura::client::ActivationChangeObserver

ash/shell.cc

Index: ash/shell.cc
diff --git a/ash/shell.cc b/ash/shell.cc
index 90c62c0b85af397d0abb5a33da500ac9527fdbff..6e03f8ecc972d36648cb701c5cd542155a18d90c 100644
--- a/ash/shell.cc
+++ b/ash/shell.cc
@@ -680,6 +680,8 @@ void Shell::Init(const ShellInitParams& init_params) {
       new ::wm::FocusController(focus_rules);
   focus_client_.reset(focus_controller);
   activation_client_ = focus_controller;
+  // TODO(sky): Shell should implement ActivationChangeObserver, not WmShell.
+  activation_client_->AddObserver(wm_shell_.get());

[James Cook, 2017/03/08 02:54:12]

nit: I would also add a note that this is cleaned up in WmShell (since it's odd
that the cleanup isn't in this file).

[sky, 2017/03/08 05:04:06]

I'm going to leave this as the next patch nukes it.

 
   screen_position_controller_.reset(new ScreenPositionController);