Fix video_decode_accelerator_unittest build on non-CrOS

Fix video_decode_accelerator_unittest build on non-CrOS

When running this unittest in non-CrOS systems, DisplayConfigurator
configure_display_ needs to be initialized as true as well, so the Native
Display will actually be created properly at ::Init time.

BUG=none
TEST=video_decode_accelerator_unittest --vmodule=*/ui/ozone/platform/drm/*=3,
and check the Ozone DRM informational messages that creates the display

Committed: https://crrev.com/ac25812351c8753b68176cf2595912738c26e4b8
Cr-Commit-Position: refs/heads/master@{#320120}

[vignatti (out of this project), 2015-03-10 19:36:49]

Patchset #2 (id:20001) has been deleted

[vignatti (out of this project), 2015-03-10 19:37:31]

tiago.vignatti@intel.com changed reviewers:
+ derat@chromium.org, posciak@google.com

[vignatti (out of this project), 2015-03-10 19:38:25]

tiago.vignatti@intel.com changed reviewers:
+ posciak@chromium.org
- posciak@google.com

[vignatti (out of this project), 2015-03-10 19:40:04]

posciak@:
content/common/gpu/media/DEPS
content/common/gpu/media/rendering_helper.cc

derat@
ui/display/chromeos/display_configurator.h

PTAL.

[Daniel Erat, 2015-03-10 19:42:15]

lgtm for ui/display/chromeos/display_configurator.h

https://codereview.chromium.org/995103002/diff/1/ui/display/chromeos/display_...
File ui/display/chromeos/display_configurator.h (right):

https://codereview.chromium.org/995103002/diff/1/ui/display/chromeos/display_...
ui/display/chromeos/display_configurator.h:359: // be enabled.  If we aren't
running on ChromeOS, we can't assume that the
nit: mind s/ChromeOS/Chrome OS/ here if you upload another revision? pet peeve.

[vignatti (out of this project), 2015-03-10 19:45:47]

https://codereview.chromium.org/995103002/diff/1/ui/display/chromeos/display_...
File ui/display/chromeos/display_configurator.h (right):

https://codereview.chromium.org/995103002/diff/1/ui/display/chromeos/display_...
ui/display/chromeos/display_configurator.h:359: // be enabled.  If we aren't
running on ChromeOS, we can't assume that the
On 2015/03/10 19:42:15, Daniel Erat wrote:
> nit: mind s/ChromeOS/Chrome OS/ here if you upload another revision? pet
peeve.

Done.

[Pawel Osciak, 2015-03-11 12:36:50]

content/common/gpu/media lgtm

[Pawel Osciak, 2015-03-11 12:38:18]

On 2015/03/11 12:36:50, Pawel Osciak wrote:
> content/common/gpu/media lgtm

By the way, the CL description says fix on non-CrOS, but changes are under #if
defined(OS_CHROMEOS) ?

[vignatti (out of this project), 2015-03-11 18:06:09]

On 2015/03/11 12:38:18, Pawel Osciak wrote:
> On 2015/03/11 12:36:50, Pawel Osciak wrote:
> > content/common/gpu/media lgtm
> 
> By the way, the CL description says fix on non-CrOS, but changes are under #if
> defined(OS_CHROMEOS) ?

well yeah. It's probably not a good practice but we also do some Chrome OS
development on regular Linux... I mean, it worked so far fine for several tasks
and as far as we can keep it like that without much hassle then it's convenient.

[vignatti (out of this project), 2015-03-11 18:06:15]

The CQ bit was checked by tiago.vignatti@intel.com

[vignatti (out of this project), 2015-03-11 18:06:15]

The patchset sent to the CQ was uploaded after l-g-t-m from derat@chromium.org
Link to the patchset: https://codereview.chromium.org/995103002/#ps40001 (title:
"s/ChromeOS/Chrome OS/")

[commit-bot: I haz the power, 2015-03-11 18:07:19]

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

[dnicoara, 2015-03-11 18:09:13]

On 2015/03/11 18:06:09, vignatti wrote:
> On 2015/03/11 12:38:18, Pawel Osciak wrote:
> > On 2015/03/11 12:36:50, Pawel Osciak wrote:
> > > content/common/gpu/media lgtm
> > 
> > By the way, the CL description says fix on non-CrOS, but changes are under
#if
> > defined(OS_CHROMEOS) ?
> 
> well yeah. It's probably not a good practice but we also do some Chrome OS
> development on regular Linux... I mean, it worked so far fine for several
tasks
> and as far as we can keep it like that without much hassle then it's
convenient.

The confusion is coming from the word "system" in the description which might
mean compiling without OS_CHROMEOS. You're really doing a CHROMEOS build but
running on a non-CrOS device.

[oshima, 2015-03-11 18:13:50]

oshima@chromium.org changed reviewers:
+ oshima@chromium.org

[oshima, 2015-03-11 18:13:51]

just fyi:

video_decode_accelerator_unittest is not covered by CQ bots right now. see
crbug.com/461437

[commit-bot: I haz the power, 2015-03-11 19:44:57]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-03-11 19:46:14]

Patchset 2 (id:??) landed as
https://crrev.com/ac25812351c8753b68176cf2595912738c26e4b8
Cr-Commit-Position: refs/heads/master@{#320120}