Remove redundant IsFullscreenAppListEnabled()

Remove redundant IsFullscreenAppListEnabled()

Call featuers::IsFullscreenAppListEnabled() once and cache it.

This is a follow up CL for issue 2938563002.

BUG=732067
TEST="Add --enable-features=EnableFullscreenAppList to see the changes"

Review-Url: https://codereview.chromium.org/2938953003
Cr-Commit-Position: refs/heads/master@{#479829}
Committed: https://chromium.googlesource.com/chromium/src/+/4b7bd924b5d4c938a1df51270fa7bd74e632f66a

[weidongg, 2017-06-15 19:38:52]

weidongg@chromium.org changed reviewers:
+ newcomer@chromium.org, xiyuan@chromium.org

[weidongg, 2017-06-15 19:38:53]

This is a follow up CL to https://codereview.chromium.org/2938563002/,
as that CL has been checked in already.

[xiyuan, 2017-06-15 19:40:05]

lgtm

https://codereview.chromium.org/2938953003/diff/1/ui/app_list/views/apps_grid...
File ui/app_list/views/apps_grid_view.h (right):

https://codereview.chromium.org/2938953003/diff/1/ui/app_list/views/apps_grid...
ui/app_list/views/apps_grid_view.h:550: bool is_fullscreen_app_list_enabled_;
nit: const bool since we don't expect to change dynamically.

[weidongg, 2017-06-15 19:40:21]

The CQ bit was checked by weidongg@chromium.org

[commit-bot: I haz the power, 2017-06-15 19:40:52]

CQ is trying da patch.

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

[newcomer, 2017-06-15 19:47:21]

lgtm

lgtm

[newcomer, 2017-06-15 19:47:24]

lgtm

lgtm

lgtm

[newcomer, 2017-06-15 19:47:27]

lgtm

[newcomer, 2017-06-15 19:47:27]

lgtm

[weidongg, 2017-06-15 20:23:14]

The CQ bit was unchecked by weidongg@chromium.org

[weidongg, 2017-06-15 20:30:42]

Thanks for the review.

https://codereview.chromium.org/2938953003/diff/1/ui/app_list/views/apps_grid...
File ui/app_list/views/apps_grid_view.h (right):

https://codereview.chromium.org/2938953003/diff/1/ui/app_list/views/apps_grid...
ui/app_list/views/apps_grid_view.h:550: bool is_fullscreen_app_list_enabled_;
On 2017/06/15 19:40:05, xiyuan wrote:
> nit: const bool since we don't expect to change dynamically.

Done.

[weidongg, 2017-06-15 20:30:46]

The CQ bit was checked by weidongg@chromium.org

[weidongg, 2017-06-15 20:30:47]

The patchset sent to the CQ was uploaded after l-g-t-m from xiyuan@chromium.org,
newcomer@chromium.org
Link to the patchset: https://codereview.chromium.org/2938953003/#ps20001
(title: "bool->const bool")

[commit-bot: I haz the power, 2017-06-15 20:31:32]

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-06-15 21:09:42]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1497558646618690,
"parent_rev": "f5036c22dc2365b355bd9e97164c105aadea9c17", "commit_rev":
"4b7bd924b5d4c938a1df51270fa7bd74e632f66a"}

[commit-bot: I haz the power, 2017-06-15 21:09:53]

Description was changed from

==========
Remove redundant IsFullscreenAppListEnabled()

Call featuers::IsFullscreenAppListEnabled() once and cache it.

This is a follow up CL for issue 2938563002.

BUG=732067
TEST="Add --enable-features=EnableFullscreenAppList to see the changes"
==========

to

==========
Remove redundant IsFullscreenAppListEnabled()

Call featuers::IsFullscreenAppListEnabled() once and cache it.

This is a follow up CL for issue 2938563002.

BUG=732067
TEST="Add --enable-features=EnableFullscreenAppList to see the changes"

Review-Url: https://codereview.chromium.org/2938953003
Cr-Commit-Position: refs/heads/master@{#479829}
Committed:
https://chromium.googlesource.com/chromium/src/+/4b7bd924b5d4c938a1df51270fa7...
==========

[commit-bot: I haz the power, 2017-06-15 21:09:55]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/4b7bd924b5d4c938a1df51270fa7...