Refactor the home card structure and introduce animation.

Refactor the home card structure and introduce animation.

This CL includes the following refactorings:
- minimized home is simply a layer instead of views
- moves the mouse click handler of minimized home to home card impl
- home card now has the same size of the screen. Changing state is
  now simply changing its position.

Then schedule the animation between BOTTOM state and CENTERED state.

BUG=398062
R=sadrul@chromium.org, oshima@chromium.org
TEST=manually on remote desktop

Committed: https://crrev.com/30e0231395281e1275f0ebfd09b31605a2ecf4cd
Cr-Commit-Position: refs/heads/master@{#292092}

[sadrul, 2014-08-21 21:30:30]

Quick question:

> - home card now has the same size of the screen. Changing
> state is now simply changing its position.

When the home-card is in the bottom state, is it possible to interact with the
rest of the system (e.g. the windows in overview mode, or just the windows)?

[Jun Mukai, 2014-08-21 21:34:33]

On 2014/08/21 21:30:30, sadrul wrote:
> Quick question:
> 
> > - home card now has the same size of the screen. Changing
> > state is now simply changing its position.
> 
> When the home-card is in the bottom state, is it possible to interact with the
> rest of the system (e.g. the windows in overview mode, or just the windows)?

Yes. In the bottom state, the home card size is full-screen, but the y-position
is at the 100px from bottom, most of the window is off-screen.

[oshima, 2014-08-21 22:58:34]

sadrul's review will be enough. removing myself.

[Jun Mukai, 2014-08-21 23:00:34]

Noticed this CL doesn't work well with screen rotation because minimized home is
not repositioned. Fixed this issue in patchset 5.

[Jun Mukai, 2014-08-22 19:50:52]

sadrul, could you review this?

[Jun Mukai, 2014-08-22 22:25:30]

noticed that SchedulePaint() at Layout() makes the animation laggy, so added
another layer for the background.

PTAL.

[Jun Mukai, 2014-08-25 18:17:37]

Sadrul, can you take a look at this CL?

[sadrul, 2014-08-26 15:47:26]

https://codereview.chromium.org/478293004/diff/140001/athena/home/minimized_h...
File athena/home/minimized_home.cc (right):

https://codereview.chromium.org/478293004/diff/140001/athena/home/minimized_h...
athena/home/minimized_home.cc:20: MinimizedHomePainter(ui::Layer* layer) {
explicit

https://codereview.chromium.org/478293004/diff/140001/athena/home/minimized_h...
File athena/home/minimized_home.h (right):

https://codereview.chromium.org/478293004/diff/140001/athena/home/minimized_h...
athena/home/minimized_home.h:16: scoped_ptr<ui::LayerOwner>
CreateMinimizedHome();
I think you should just return a Layer here.

[Jun Mukai, 2014-08-26 17:21:33]

https://codereview.chromium.org/478293004/diff/140001/athena/home/minimized_h...
File athena/home/minimized_home.cc (right):

https://codereview.chromium.org/478293004/diff/140001/athena/home/minimized_h...
athena/home/minimized_home.cc:20: MinimizedHomePainter(ui::Layer* layer) {
On 2014/08/26 15:47:26, sadrul wrote:
> explicit

Done.

https://codereview.chromium.org/478293004/diff/140001/athena/home/minimized_h...
File athena/home/minimized_home.h (right):

https://codereview.chromium.org/478293004/diff/140001/athena/home/minimized_h...
athena/home/minimized_home.h:16: scoped_ptr<ui::LayerOwner>
CreateMinimizedHome();
On 2014/08/26 15:47:26, sadrul wrote:
> I think you should just return a Layer here.

I need the LayerDelegate to draw the content of minimized home, and the layer
delegate needs to be owned by someone else. Thus I created the minimized home
layer and then returns its owner (which is actually the LayerDelegate), and
caller will own both the layer and the delegate.

Do you think this needs to be fixed?  It seems doing similarly as aura::Window.
As I reviewed aura::Window interface, I found ignore_events() field.  This could
be implemented by aura::Window with ignore_events field.

[sadrul, 2014-08-26 20:11:18]

LGTM

https://codereview.chromium.org/478293004/diff/140001/athena/home/minimized_h...
File athena/home/minimized_home.h (right):

https://codereview.chromium.org/478293004/diff/140001/athena/home/minimized_h...
athena/home/minimized_home.h:16: scoped_ptr<ui::LayerOwner>
CreateMinimizedHome();
On 2014/08/26 17:21:33, Jun Mukai wrote:
> On 2014/08/26 15:47:26, sadrul wrote:
> > I think you should just return a Layer here.
> 
> I need the LayerDelegate to draw the content of minimized home, and the layer
> delegate needs to be owned by someone else. Thus I created the minimized home
> layer and then returns its owner (which is actually the LayerDelegate), and
> caller will own both the layer and the delegate.

Ah, I see.

> 
> Do you think this needs to be fixed?  It seems doing similarly as
aura::Window.
> As I reviewed aura::Window interface, I found ignore_events() field.  This
could
> be implemented by aura::Window with ignore_events field.

https://codereview.chromium.org/478293004/diff/180001/athena/home/home_card_i...
File athena/home/home_card_impl.cc (right):

https://codereview.chromium.org/478293004/diff/180001/athena/home/home_card_i...
athena/home/home_card_impl.cc:74: explicit HomeCardLayoutManager()
You don't need 'explicit' here.

[Jun Mukai, 2014-08-26 20:48:45]

https://codereview.chromium.org/478293004/diff/180001/athena/home/home_card_i...
File athena/home/home_card_impl.cc (right):

https://codereview.chromium.org/478293004/diff/180001/athena/home/home_card_i...
athena/home/home_card_impl.cc:74: explicit HomeCardLayoutManager()
On 2014/08/26 20:11:18, sadrul wrote:
> You don't need 'explicit' here.

Done.

[Jun Mukai, 2014-08-26 20:48:56]

The CQ bit was checked by mukai@chromium.org

[commit-bot: I haz the power, 2014-08-26 20:50:13]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mukai@chromium.org/478293004/220001

[commit-bot: I haz the power, 2014-08-27 03:37:33]

Committed patchset #12 (220001) as e205d4af6e1dc98aad72b6dbdda62929d18c62c2

[commit-bot: I haz the power, 2014-09-10 02:49:34]

Patchset 12 (id:??) landed as
https://crrev.com/30e0231395281e1275f0ebfd09b31605a2ecf4cd
Cr-Commit-Position: refs/heads/master@{#292092}