Adds AppListDelegate to AppList that is notified on visibility changes

Adds AppListDelegate to AppList that is notified on visibility changes

For mash chrome shows the app-list. Mash needs to inform the
AppListButton when the visibility changes otherwise the ink drop
animations checks. To address this ash installs an AppListDelegate
that is notified on visibility changes so that it can update the
button.

BUG=681168
TEST=none
R=xiyuan@chromium.org

Review-Url: https://codereview.chromium.org/2647833003
Cr-Commit-Position: refs/heads/master@{#445111}
Committed: https://chromium.googlesource.com/chromium/src/+/15ef0aa9060364567a6cf2abd20f6d419619cc83

[sky, 2017-01-20 04:07:33]

Description was changed from

==========
Adds AppListDelegate to AppList that is notified on visibility changes

For mash chrome shows the app-list. Mash needs to inform the
AppListButton when the visibility changes otherwise the ink drop
animations checks. To address this ash installs an AppListDelegate
that is notified on visibility changes so that it can update the
button.

BUG=681168
TEST=none
R=xiyuan@chromium.org
==========

to

==========
Adds AppListDelegate to AppList that is notified on visibility changes

For mash chrome shows the app-list. Mash needs to inform the
AppListButton when the visibility changes otherwise the ink drop
animations checks. To address this ash installs an AppListDelegate
that is notified on visibility changes so that it can update the
button.

BUG=681168
TEST=none
R=xiyuan@chromium.org
==========

[sky, 2017-01-20 04:07:34]

sky@chromium.org changed reviewers:
+ tsepez@chromium.org

[sky, 2017-01-20 04:07:48]

tsepez: mojom
xiyuan: the rest

[sky, 2017-01-20 04:07:50]

The CQ bit was checked by sky@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-20 04:08:06]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-20 04:49:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-20 04:49:45]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sky, 2017-01-20 16:59:33]

The CQ bit was checked by sky@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-20 16:59:49]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[xiyuan, 2017-01-20 17:20:49]

lgtm

https://codereview.chromium.org/2647833003/diff/40001/ui/app_list/presenter/a...
File ui/app_list/presenter/app_list.h (right):

https://codereview.chromium.org/2647833003/diff/40001/ui/app_list/presenter/a...
ui/app_list/presenter/app_list.h:22: void set_delegate(AppListDelegate*
delegate) { delegate_ = delegate; }
nit: move accessor to the last of declarations since that is a more popular
style ?

[Tom Sepez, 2017-01-20 17:41:39]

lgtm

[sky, 2017-01-20 17:45:56]

https://codereview.chromium.org/2647833003/diff/40001/ui/app_list/presenter/a...
File ui/app_list/presenter/app_list.h (right):

https://codereview.chromium.org/2647833003/diff/40001/ui/app_list/presenter/a...
ui/app_list/presenter/app_list.h:22: void set_delegate(AppListDelegate*
delegate) { delegate_ = delegate; }
On 2017/01/20 17:20:49, xiyuan wrote:
> nit: move accessor to the last of declarations since that is a more popular
> style ?

Done.

[sky, 2017-01-20 17:46:09]

The CQ bit was checked by sky@chromium.org

[sky, 2017-01-20 17:46:09]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
xiyuan@chromium.org
Link to the patchset: https://codereview.chromium.org/2647833003/#ps60001
(title: "move")

[commit-bot: I haz the power, 2017-01-20 17:46:12]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-20 18:56:03]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1484934369476600,
"parent_rev": "5fe1163de91a80f23c4a2b68ca1d1a5c53319986", "commit_rev":
"15ef0aa9060364567a6cf2abd20f6d419619cc83"}

[commit-bot: I haz the power, 2017-01-20 18:56:33]

Description was changed from

==========
Adds AppListDelegate to AppList that is notified on visibility changes

For mash chrome shows the app-list. Mash needs to inform the
AppListButton when the visibility changes otherwise the ink drop
animations checks. To address this ash installs an AppListDelegate
that is notified on visibility changes so that it can update the
button.

BUG=681168
TEST=none
R=xiyuan@chromium.org
==========

to

==========
Adds AppListDelegate to AppList that is notified on visibility changes

For mash chrome shows the app-list. Mash needs to inform the
AppListButton when the visibility changes otherwise the ink drop
animations checks. To address this ash installs an AppListDelegate
that is notified on visibility changes so that it can update the
button.

BUG=681168
TEST=none
R=xiyuan@chromium.org

Review-Url: https://codereview.chromium.org/2647833003
Cr-Commit-Position: refs/heads/master@{#445111}
Committed:
https://chromium.googlesource.com/chromium/src/+/15ef0aa9060364567a6cf2abd20f...
==========

[commit-bot: I haz the power, 2017-01-20 18:56:35]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/15ef0aa9060364567a6cf2abd20f...