Makes the maximize/restore button handling

Makes the maximize/restore button handling
snapping/minimizing. FrameMaximizeButton handles all the logic for
this. If you move the mouse the drag threshold we'll trigger to
snapping behavior. moving to the left snaps to the left half, moving
to the right the right half, and below minimizes. Uses
PhantomWindowController to render the snap position. Minimizing
highlights the launcher button. To support this I had to plumb through
getting the bounds of the launcher button.

BUG=116213
TEST=none
R=ben@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=126729

[tfarina, 2012-03-14 16:36:03]

http://codereview.chromium.org/9703026/diff/25/ui/views/controls/button/image...
File ui/views/controls/button/image_button.h (left):

http://codereview.chromium.org/9703026/diff/25/ui/views/controls/button/image...
ui/views/controls/button/image_button.h:26: virtual void SetImage(ButtonState
aState, const SkBitmap* anImage);
nit: anImage -> bitmap or image?

[tfarina, 2012-03-14 16:38:42]

http://codereview.chromium.org/9703026/diff/25/ash/wm/workspace/frame_maximiz...
File ash/wm/workspace/frame_maximize_button.h (right):

http://codereview.chromium.org/9703026/diff/25/ash/wm/workspace/frame_maximiz...
ash/wm/workspace/frame_maximize_button.h:13: class MouseEvent;
nit: no need to forward declare this, it comes from ui/views/view.h

[Ben Goodger (Google), 2012-03-14 16:46:22]

LGTM

[sky, 2012-03-14 16:47:52]

I incorporated your review feedback Thiago.

  -Scott

On Wed, Mar 14, 2012 at 9:38 AM,  <tfarina@chromium.org> wrote:
>
>
http://codereview.chromium.org/9703026/diff/25/ash/wm/workspace/frame_maximiz...
> File ash/wm/workspace/frame_maximize_button.h (right):
>
>
http://codereview.chromium.org/9703026/diff/25/ash/wm/workspace/frame_maximiz...
> ash/wm/workspace/frame_maximize_button.h:13: class MouseEvent;
> nit: no need to forward declare this, it comes from ui/views/view.h
>
> http://codereview.chromium.org/9703026/