Add app_list_demo, a standalone executable for testing the app list UI.

Add app_list_demo, a standalone executable for testing the app list UI.

This will facilitate development of the out-of-process UI compositor for
MacViews by providing a views example that uses UI layers and composited
animations.

It also gives app list developers a target to ease development of
upcoming changes, without compiling all of Chromium and manually
installing two dozen apps from the webstore.

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

[tapted, 2014-05-26 06:40:06]

Hi Chris - could you take a look?

[calamity, 2014-05-27 03:32:10]

Looks awesome, lgtm.

https://codereview.chromium.org/281383012/diff/100001/ui/app_list/views/app_l...
File ui/app_list/views/app_list_demo.cc (right):

https://codereview.chromium.org/281383012/diff/100001/ui/app_list/views/app_l...
ui/app_list/views/app_list_demo.cc:57: view_ = new app_list::AppListView(this);
Note here that this is where the DemoAppListViewDelegate becomes owned.

[calamity, 2014-05-27 03:37:33]

On 2014/05/27 03:32:10, calamity wrote:
> Looks awesome, lgtm.
> 
>
https://codereview.chromium.org/281383012/diff/100001/ui/app_list/views/app_l...
> File ui/app_list/views/app_list_demo.cc (right):
> 
>
https://codereview.chromium.org/281383012/diff/100001/ui/app_list/views/app_l...
> ui/app_list/views/app_list_demo.cc:57: view_ = new
app_list::AppListView(this);
> Note here that this is where the DemoAppListViewDelegate becomes owned.

Oh, and FYI, I got some errors when compiling on Windows with namespace
conflicts around
AppListViewTestApi. Line 119 should be 'friend class
::test::AppListViewTestApi;'. OTOH, the
bots seem to be doing okay so maybe it's just me.

[tapted, 2014-05-27 06:20:07]

On 2014/05/27 03:32:10, calamity wrote:
> Oh, and FYI, I got some errors when compiling on Windows with namespace
> conflicts around
> AppListViewTestApi. Line 119 should be 'friend class
> ::test::c;'. OTOH, the
> bots seem to be doing okay so maybe it's just me.

Done.

Yeah I started hitting these after the rebase. Since there is both ::test:: and
app_list::test:: the compiler gets confused. I think we can move
test::AppListViewTestApi into app_list::test for a bit more consistency in a
follow-up.

https://codereview.chromium.org/281383012/diff/100001/ui/app_list/views/app_l...
File ui/app_list/views/app_list_demo.cc (right):

https://codereview.chromium.org/281383012/diff/100001/ui/app_list/views/app_l...
ui/app_list/views/app_list_demo.cc:57: view_ = new app_list::AppListView(this);
On 2014/05/27 03:32:10, calamity wrote:
> Note here that this is where the DemoAppListViewDelegate becomes owned.

Done. Maybe we should change the constructor to take a scoped_ptr the way Mac
does...

[tapted, 2014-05-27 06:23:11]

+ben for DEPS OWNERS

Could you please take a look for:
> Missing LGTM from OWNERS of dependencies added to DEPS:
>    '+sandbox',
>    '+ui/views_content_client',
>
> Suggested missing target path OWNERS:
>    ben@chromium.org

[Ben Goodger (Google), 2014-05-27 15:56:06]

deps lgtm

[tapted, 2014-05-27 21:38:07]

The CQ bit was checked by tapted@chromium.org

[commit-bot: I haz the power, 2014-05-27 21:39:08]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tapted@chromium.org/281383012/170001

[commit-bot: I haz the power, 2014-05-27 22:33:25]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  ios_rel_device_ninja on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device_ninja/...)

[commit-bot: I haz the power, 2014-05-27 22:49:37]

Change committed as 273063