Only compile ozone_platform_headless for headless.gn.

Only compile ozone_platform_headless for headless.gn.

Ensure that ozone_platform_headless is the only ozone platform compiled
in the sample headless configuration. Other ozone platforms are included
by default and aren't necessarily desired.

BUG=none

Committed: https://crrev.com/7dfdf188b6c9bfc4e208949a1be8af557c0264b0
Cr-Commit-Position: refs/heads/master@{#386672}

[kylechar, 2016-04-11 20:30:27]

kylechar@chromium.org changed reviewers:
+ dpranke@chromium.org

[Dirk Pranke, 2016-04-11 21:47:07]

https://codereview.chromium.org/1876273002/diff/1/build/args/headless.gn
File build/args/headless.gn (right):

https://codereview.chromium.org/1876273002/diff/1/build/args/headless.gn#newc...
build/args/headless.gn:13: ozone_platform_headless = true
why do we need both lines 12 and 13? That seems kinda redundant ...

[kylechar, 2016-04-11 22:34:43]

https://codereview.chromium.org/1876273002/diff/1/build/args/headless.gn
File build/args/headless.gn (right):

https://codereview.chromium.org/1876273002/diff/1/build/args/headless.gn#newc...
build/args/headless.gn:13: ozone_platform_headless = true
On 2016/04/11 at 21:47:07, Dirk Pranke wrote:
> why do we need both lines 12 and 13? That seems kinda redundant ...

The first one tells Ozone to use the headless platform at runtime if no platform
is specified via --ozone-platform=<platform>. The second one says to compile
headless so it's available at runtime.

I think Ozone just grabs the first available platform in the list if
ozone_platform and --ozone-platform=<platform> are both omitted, so its
technically not necessary, but if someone were to add other platforms to this
config it would ensure headless is what gets selected by default.

[Dirk Pranke, 2016-04-11 22:55:04]

gotcha, I figured it was something like that.

lgtm.

[kylechar, 2016-04-12 13:08:13]

The CQ bit was checked by kylechar@chromium.org

[commit-bot: I haz the power, 2016-04-12 13:08:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1876273002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1876273002/1

[commit-bot: I haz the power, 2016-04-12 13:47:20]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-12 13:48:32]

Description was changed from

==========
Only compile ozone_platform_headless for headless.gn.

Ensure that ozone_platform_headless is the only ozone platform compiled
in the sample headless configuration. Other ozone platforms are included
by default and aren't necessarily desired.

BUG=none
==========

to

==========
Only compile ozone_platform_headless for headless.gn.

Ensure that ozone_platform_headless is the only ozone platform compiled
in the sample headless configuration. Other ozone platforms are included
by default and aren't necessarily desired.

BUG=none

Committed: https://crrev.com/7dfdf188b6c9bfc4e208949a1be8af557c0264b0
Cr-Commit-Position: refs/heads/master@{#386672}
==========

[commit-bot: I haz the power, 2016-04-12 13:48:33]

Patchset 1 (id:??) landed as
https://crrev.com/7dfdf188b6c9bfc4e208949a1be8af557c0264b0
Cr-Commit-Position: refs/heads/master@{#386672}