Move ContentsView::NamedPage to AppListModel::ShowState.

Move ContentsView::NamedPage to AppListModel::ShowState.

This CL moves the NamedPage enum from ContentsView into AppListModel where
it will be used as a more generic value that represents the state of the
app list.

This is in preparation of notifying code across the chrome/ DEPS boundary
of changes to the app list's state.

BUG=425444
Committed: https://crrev.com/060d7dc875cc41f46ab264181d1d622d02c3c8b7
Cr-Commit-Position: refs/heads/master@{#301315}

[calamity, 2014-10-24 00:27:28]

Patchset #1 (id:1) has been deleted

[calamity, 2014-10-24 02:22:31]

calamity@chromium.org changed reviewers:
+ mgiuca@chromium.org

[calamity, 2014-10-24 02:22:50]

Patchset #1 (id:20001) has been deleted

[Matt Giuca, 2014-10-24 03:23:48]

lgtm with nits.

https://codereview.chromium.org/675053002/diff/40001/ui/app_list/app_list_mod...
File ui/app_list/app_list_model.h (right):

https://codereview.chromium.org/675053002/diff/40001/ui/app_list/app_list_mod...
ui/app_list/app_list_model.h:43: SHOW_APPS,
nit: These should be SHOW_STATE_foo.

https://codereview.chromium.org/675053002/diff/40001/ui/app_list/app_list_mod...
ui/app_list/app_list_model.h:45: SHOW_START,
Should there be a SHOW_STATE_CUSTOM? (Or is that coming later... whatever, I
won't block on it.)

[calamity, 2014-10-24 06:26:35]

https://codereview.chromium.org/675053002/diff/40001/ui/app_list/app_list_mod...
File ui/app_list/app_list_model.h (right):

https://codereview.chromium.org/675053002/diff/40001/ui/app_list/app_list_mod...
ui/app_list/app_list_model.h:43: SHOW_APPS,
On 2014/10/24 03:23:48, Matt Giuca wrote:
> nit: These should be SHOW_STATE_foo.

renamed to State.

https://codereview.chromium.org/675053002/diff/40001/ui/app_list/app_list_mod...
ui/app_list/app_list_model.h:45: SHOW_START,
On 2014/10/24 03:23:48, Matt Giuca wrote:
> Should there be a SHOW_STATE_CUSTOM? (Or is that coming later... whatever, I
> won't block on it.)

Yeah, I had it here before. We'll define it when we need it.

[calamity, 2014-10-24 06:29:53]

The CQ bit was checked by calamity@chromium.org

[commit-bot: I haz the power, 2014-10-24 06:30:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/675053002/60001

[commit-bot: I haz the power, 2014-10-24 06:33:08]

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

[commit-bot: I haz the power, 2014-10-24 06:33:09]

Try jobs failed on following builders:
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/73026)
  android_arm64_dbg_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_arm64_d...)
  android_clang_dbg_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...)
  mac_chromium_rel on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel/...)
  win8_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)
  win_chromium_x64_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[calamity, 2014-10-27 03:16:51]

The CQ bit was checked by calamity@chromium.org

[commit-bot: I haz the power, 2014-10-27 03:17:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/675053002/100001

[commit-bot: I haz the power, 2014-10-27 04:58:15]

Committed patchset #4 (id:100001)

[commit-bot: I haz the power, 2014-10-27 04:59:24]

Patchset 4 (id:??) landed as
https://crrev.com/060d7dc875cc41f46ab264181d1d622d02c3c8b7
Cr-Commit-Position: refs/heads/master@{#301315}