mus: Fix activation cycle direction.

components/mus/ws/focus_controller.h

 // Copyright 2015 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.
 
 #ifndef COMPONENTS_MUS_WS_FOCUS_CONTROLLER_H_
 #define COMPONENTS_MUS_WS_FOCUS_CONTROLLER_H_
 
 #include "base/macros.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/observer_list.h"
 #include "components/mus/ws/server_window_drawn_tracker_observer.h"
+#include "components/mus/ws/server_window_observer.h"
 
 namespace mus {
 
 namespace ws {
 
 class FocusControllerDelegate;
 class FocusControllerObserver;
 class ServerWindow;
 class ServerWindowDrawnTracker;
+struct WindowId;
 
 // Describes the source of the change.
 enum class FocusControllerChangeSource {
   EXPLICIT,
   DRAWN_STATE_CHANGED,
 };
 
 // Tracks the focused window. Focus is moved to another window when the drawn
 // state of the focused window changes.
-class FocusController : public ServerWindowDrawnTrackerObserver {
+class FocusController : public ServerWindowDrawnTrackerObserver,
+                        public ServerWindowObserver {
  public:
   FocusController(FocusControllerDelegate* delegate, ServerWindow* root);
   ~FocusController() override;
 
   // Sets the focused window. Does nothing if |window| is currently focused.
   // This does not notify the delegate.
   void SetFocusedWindow(ServerWindow* window);
   ServerWindow* GetFocusedWindow();
 
   // Moves activation to the next activatable window.
   void ActivateNextWindow();
 
   void AddObserver(FocusControllerObserver* observer);
   void RemoveObserver(FocusControllerObserver* observer);
 
  private:
-  void SetActiveWindow(ServerWindow* window);
+  enum class ActivationChangeReason {
+    UNKNONW,
+    CYCLE,  // Activation changed because of ActivateNextWindow().
+    FOCUS,  // Focus change required a different window to be activated.
+    DRAWN_STATE_CHANGED,  // Active window was hidden or destroyed.
+  };
+  void SetActiveWindow(ServerWindow* window, ActivationChangeReason reason);
 
   // Returns whether |window| can be focused or activated.
   bool CanBeFocused(ServerWindow* window) const;
   bool CanBeActivated(ServerWindow* window) const;
 
   // Returns the closest activatable ancestor of |window|. Returns nullptr if
   // there is no such ancestor.
   ServerWindow* GetActivatableAncestorOf(ServerWindow* window) const;
 
   // Implementation of SetFocusedWindow().
   void SetFocusedWindowImpl(FocusControllerChangeSource change_source,
                             ServerWindow* window);
 
+  void ClearCycleWindows();
+
   // ServerWindowDrawnTrackerObserver:
   void OnDrawnStateWillChange(ServerWindow* ancestor,
                               ServerWindow* window,
                               bool is_drawn) override;
   void OnDrawnStateChanged(ServerWindow* ancestor,
                            ServerWindow* window,
                            bool is_drawn) override;
 
+  // ServerWindowObserver:
+  void OnWillDestroyWindow(ServerWindow* window) override;
+
   FocusControllerDelegate* delegate_;
 
   ServerWindow* root_;
   ServerWindow* focused_window_;
   ServerWindow* active_window_;
+  // Tracks what caused |active_window_| to be activated.
+  ActivationChangeReason activation_reason_;
+
+  std::vector<ServerWindow*> cycle_windows_;

[sky, 2016/01/06 21:02:39]

It's not obvious what this is, or how it changes based on the name (nor do I
think you could come up with a name that would convey it). Please add a
description.

[sadrul, 2016/01/08 20:40:26]

Done.

 
   base::ObserverList<FocusControllerObserver> observers_;
 
   // Keeps track of the visibility of the focused and active window.
   scoped_ptr<ServerWindowDrawnTracker> drawn_tracker_;
 
   DISALLOW_COPY_AND_ASSIGN(FocusController);
 };
 
 }  // namespace ws
 
 }  // namespace mus
 
 #endif  // COMPONENTS_MUS_WS_FOCUS_CONTROLLER_H_