Refactor PhantomWindowController so that the PhantomWindowController owns the phantom window

ash/wm/workspace/phantom_window_controller.h

 // 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.
 
 #ifndef ASH_WM_WORKSPACE_PHANTOM_WINDOW_CONTROLLER_H_
 #define ASH_WM_WORKSPACE_PHANTOM_WINDOW_CONTROLLER_H_
 
 #include "ash/ash_export.h"
 #include "base/basictypes.h"
 #include "base/gtest_prod_util.h"
 #include "base/memory/scoped_ptr.h"
-#include "ui/gfx/animation/animation_delegate.h"
 #include "ui/gfx/rect.h"
 
 namespace aura {
 class Window;
 }
 
-namespace gfx {
-class SlideAnimation;
-}
-
 namespace views {
 class Widget;
 }
 
 namespace ash {
 namespace internal {
 
 // PhantomWindowController is responsible for showing a phantom representation
-// of a window. It's used used during dragging a window to show a snap location.
-class ASH_EXPORT PhantomWindowController : public gfx::AnimationDelegate {
+// of a window. It's used used to show a preview of how snapping or docking a
+// window will affect the window's bounds.
+class ASH_EXPORT PhantomWindowController {
  public:
   explicit PhantomWindowController(aura::Window* window);
+
+  // Hides the phantom window immediately.
   virtual ~PhantomWindowController();
 
-  // Bounds last passed to Show().
-  const gfx::Rect& bounds_in_screen() const { return bounds_in_screen_; }
-
   // Animates the phantom window towards |bounds_in_screen|.
   // Creates two (if start bounds intersect any root window other than the
   // root window that matches the target bounds) or one (otherwise) phantom
   // widgets to display animated rectangle in each root.
   // This does not immediately show the window.
   void Show(const gfx::Rect& bounds_in_screen);
 
-  // Hides the phantom.
+  // Starts fading out the phantom window.
+  void StartFadeOut();
+
+  // Hides the phantom window without any animation.
   void Hide();
 
   // Returns true if the phantom is showing.
   bool IsShowing() const;
 
   // If set, the phantom window is stacked below this window, otherwise it
   // is stacked above the window passed to the constructor.
   void set_phantom_below_window(aura::Window* phantom_below_window) {
     phantom_below_window_ = phantom_below_window;
   }
 
-  // gfx::AnimationDelegate overrides:
-  virtual void AnimationProgressed(const gfx::Animation* animation) OVERRIDE;
-
  private:
   FRIEND_TEST_ALL_PREFIXES(WorkspaceWindowResizerTest, PhantomWindowShow);
 
   // Creates, shows and returns a phantom widget at |bounds|
-  // with kShellWindowId_ShelfContainer in |root_window| as a parent.
-  views::Widget* CreatePhantomWidget(aura::Window* root_window,
-                                     const gfx::Rect& bounds_in_screen);
+  // with kShellWindowId_ShelfContainer in |root_window| as a parent. The caller
+  // takes ownership of the returned widget.
+  views::Widget* CreatePhantomWidget(
+      aura::Window* root_window,
+      const gfx::Rect& bounds_in_screen) WARN_UNUSED_RESULT;
+
+  // Starts fading the phantom window in or out.
+  void StartFade(bool fade_in);
 
   // Window the phantom is placed beneath.
   aura::Window* window_;
 
   // If set, the phantom window should get stacked below this window.
   aura::Window* phantom_below_window_;
 
-  // Initially the bounds of |window_| (in screen coordinates).
-  // Each time Show() is invoked |start_bounds_| is then reset to the bounds of
-  // |phantom_widget_| and |bounds_| is set to the value passed into Show().
-  // The animation animates between these two values.
-  gfx::Rect start_bounds_;
+  // Target visibility of the animation.
+  bool target_visibility_;
 
   // Target bounds of the animation in screen coordinates.
-  gfx::Rect bounds_in_screen_;
+  gfx::Rect target_bounds_in_screen_;
 
   // The primary phantom representation of the window. It is parented by the
   // root window matching the target bounds.
-  views::Widget* phantom_widget_;
+  scoped_ptr<views::Widget> phantom_widget_;
 
   // If the animation starts on another display, this is the secondary phantom
   // representation of the window used on the initial display, otherwise this is
   // NULL. This allows animation to progress from one display into the other.

[Mr4D (OOO till 08-26), 2013/12/17 22:55:20]

Somehow the name is ... odd. What about naming one target_phantom_widget and the
other start_phantom_widget? 

-  views::Widget* phantom_widget_start_;
-
-  // Used to transition the bounds.
-  scoped_ptr<gfx::SlideAnimation> animation_;
+  scoped_ptr<views::Widget> phantom_widget_start_;
 
   DISALLOW_COPY_AND_ASSIGN(PhantomWindowController);
 };
 
 }  // namespace internal
 }  // namespace ash
 
 #endif  // ASH_WM_WORKSPACE_PHANTOM_WINDOW_CONTROLLER_H_