Add a unit test for the experimental app list.

Add a unit test for the experimental app list.

This CL adds a unit test for the experimental app list that instantiates
the app list in experimental mode and performs a series of very basic
checks.

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

[tapted, 2014-05-22 05:01:27]

https://codereview.chromium.org/291653003/diff/1/ui/app_list/views/app_list_v...
File ui/app_list/views/app_list_view_unittest.cc (right):

https://codereview.chromium.org/291653003/diff/1/ui/app_list/views/app_list_v...
ui/app_list/views/app_list_view_unittest.cc:245: if (test_type_ != EXPERIMENTAL)
Can it still PopulateApps, and we just check the start_page_view() is NULL /
invisible before returning?

https://codereview.chromium.org/291653003/diff/1/ui/app_list/views/app_list_v...
ui/app_list/views/app_list_view_unittest.cc:251: 
EXPECT_EQ(2, pagination_model_.total_pages()); ?

https://codereview.chromium.org/291653003/diff/1/ui/app_list/views/app_list_v...
ui/app_list/views/app_list_view_unittest.cc:265: 
EXPECT_FALSE(main_view->contents_view()->start_page_view()->visible())?

or something to that effect, since it might just be offscreen.. hm.

[calamity, 2014-05-23 04:10:23]

https://codereview.chromium.org/291653003/diff/1/ui/app_list/views/app_list_v...
File ui/app_list/views/app_list_view_unittest.cc (right):

https://codereview.chromium.org/291653003/diff/1/ui/app_list/views/app_list_v...
ui/app_list/views/app_list_view_unittest.cc:245: if (test_type_ != EXPERIMENTAL)
On 2014/05/22 05:01:27, tapted wrote:
> Can it still PopulateApps, and we just check the start_page_view() is NULL /
> invisible before returning?

Done.

https://codereview.chromium.org/291653003/diff/1/ui/app_list/views/app_list_v...
ui/app_list/views/app_list_view_unittest.cc:251: 
On 2014/05/22 05:01:27, tapted wrote:
> EXPECT_EQ(2, pagination_model_.total_pages()); ?

Is this worth rechecking given that it's already checked by the other tests?
There'd be no functional difference up to this point.

https://codereview.chromium.org/291653003/diff/1/ui/app_list/views/app_list_v...
ui/app_list/views/app_list_view_unittest.cc:265: 
On 2014/05/22 05:01:27, tapted wrote:
> EXPECT_FALSE(main_view->contents_view()->start_page_view()->visible())?
> 
> or something to that effect, since it might just be offscreen.. hm.

Yeahhhhhh it's just offscreen. =(

Made it check the origin of the views which are laid out to 0, 0 when the views
are showing.

[tapted, 2014-05-23 06:32:51]

niice - lgtm

[calamity, 2014-05-26 04:42:08]

The CQ bit was checked by calamity@chromium.org

[commit-bot: I haz the power, 2014-05-26 04:42:59]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/calamity@chromium.org/291653003/30001

[commit-bot: I haz the power, 2014-05-26 20:06:34]

Change committed as 272871