[Ozone-Demo] Add support for NativeDisplayDelegate

[Ozone-Demo] Add support for NativeDisplayDelegate

If a NativeDisplayDelegate is provided, then use it to configure the
displays and show a DemoWindow on each display.

BUG=none

Committed: https://crrev.com/655c4e624ca9257fec92be97c2e6424b73c0c92d
Cr-Commit-Position: refs/heads/master@{#315798}

[dnicoara, 2015-01-13 21:47:51]

dnicoara@chromium.org changed reviewers:
+ alexst@chromium.org

[dnicoara, 2015-01-13 21:47:52]

PTAL

[dnicoara, 2015-01-14 15:07:01]

Removed static declaration as per offline discussion.

[alexst (slow to review), 2015-02-11 15:36:05]

https://codereview.chromium.org/805003002/diff/140001/ui/ozone/demo/gl_render...
File ui/ozone/demo/gl_renderer.cc (right):

https://codereview.chromium.org/805003002/diff/140001/ui/ozone/demo/gl_render...
ui/ozone/demo/gl_renderer.cc:47: context_->MakeCurrent(surface_.get());
Why do we need this on every render frame?

https://codereview.chromium.org/805003002/diff/140001/ui/ozone/demo/ozone_dem...
File ui/ozone/demo/ozone_demo.cc (right):

https://codereview.chromium.org/805003002/diff/140001/ui/ozone/demo/ozone_dem...
ui/ozone/demo/ozone_demo.cc:217: return nullptr;
Every switch case is handled, I don't think you need this here.

https://codereview.chromium.org/805003002/diff/140001/ui/ozone/demo/ozone_dem...
ui/ozone/demo/ozone_demo.cc:264: void WindowManager::OnGetDisplays(
OnDisplaysAquired or soemthing? OnGet.. feels odd somehow.

[dnicoara, 2015-02-11 16:46:38]

https://codereview.chromium.org/805003002/diff/140001/ui/ozone/demo/gl_render...
File ui/ozone/demo/gl_renderer.cc (right):

https://codereview.chromium.org/805003002/diff/140001/ui/ozone/demo/gl_render...
ui/ozone/demo/gl_renderer.cc:47: context_->MakeCurrent(surface_.get());
On 2015/02/11 15:36:05, alexst wrote:
> Why do we need this on every render frame?

If we have multiple renderers, each will have its own context and surface. This
needs to make sure it is rendering in the correct surface.

https://codereview.chromium.org/805003002/diff/140001/ui/ozone/demo/ozone_dem...
File ui/ozone/demo/ozone_demo.cc (right):

https://codereview.chromium.org/805003002/diff/140001/ui/ozone/demo/ozone_dem...
ui/ozone/demo/ozone_demo.cc:217: return nullptr;
On 2015/02/11 15:36:05, alexst wrote:
> Every switch case is handled, I don't think you need this here.

The CrOS build complains without this.

https://codereview.chromium.org/805003002/diff/140001/ui/ozone/demo/ozone_dem...
ui/ozone/demo/ozone_demo.cc:264: void WindowManager::OnGetDisplays(
On 2015/02/11 15:36:05, alexst wrote:
> OnDisplaysAquired or soemthing? OnGet.. feels odd somehow.

Done.

[alexst (slow to review), 2015-02-11 16:49:46]

lgtm

[dnicoara, 2015-02-11 18:03:21]

The CQ bit was checked by dnicoara@chromium.org

[commit-bot: I haz the power, 2015-02-11 18:03:51]

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

[commit-bot: I haz the power, 2015-02-11 18:26:03]

Committed patchset #9 (id:160001)

[commit-bot: I haz the power, 2015-02-11 18:27:04]

Patchset 9 (id:??) landed as
https://crrev.com/655c4e624ca9257fec92be97c2e6424b73c0c92d
Cr-Commit-Position: refs/heads/master@{#315798}