Move libdrm target behind conditional

Move libdrm target behind conditional

On embedded platforms the library may not be available. As-is GYP will
try running pkg-config early on even if none of the Ozone dependencies
require libdrm.

BUG=none

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

[dnicoara, 2014-07-29 21:36:51]

PTAL

[spang, 2014-07-29 21:43:03]

lgtm

https://codereview.chromium.org/427853004/diff/20001/build/linux/system.gyp
File build/linux/system.gyp (right):

https://codereview.chromium.org/427853004/diff/20001/build/linux/system.gyp#n...
build/linux/system.gyp:422: 'target_name': 'dridrm',
while you're here, fix the name.

https://codereview.chromium.org/427853004/diff/20001/ui/ozone/platform/dri/BU...
File ui/ozone/platform/dri/BUILD.gn (right):

https://codereview.chromium.org/427853004/diff/20001/ui/ozone/platform/dri/BU...
ui/ozone/platform/dri/BUILD.gn:8: if (ozone_platform_dri || ozone_platform_gbm)
{
is this condition really needed? I think just moving it here should help.

[dnicoara, 2014-07-29 21:50:49]

Thanks!

https://codereview.chromium.org/427853004/diff/20001/build/linux/system.gyp
File build/linux/system.gyp (right):

https://codereview.chromium.org/427853004/diff/20001/build/linux/system.gyp#n...
build/linux/system.gyp:422: 'target_name': 'dridrm',
On 2014/07/29 21:43:03, spang wrote:
> while you're here, fix the name.

Done.

https://codereview.chromium.org/427853004/diff/20001/ui/ozone/platform/dri/BU...
File ui/ozone/platform/dri/BUILD.gn (right):

https://codereview.chromium.org/427853004/diff/20001/ui/ozone/platform/dri/BU...
ui/ozone/platform/dri/BUILD.gn:8: if (ozone_platform_dri || ozone_platform_gbm)
{
On 2014/07/29 21:43:03, spang wrote:
> is this condition really needed? I think just moving it here should help.

Done.

[dnicoara, 2014-07-29 21:51:30]

+ brettw@ for build/config OWNERS

[brettw, 2014-07-30 22:25:01]

lgtm

[dnicoara, 2014-07-31 01:16:11]

The CQ bit was checked by dnicoara@chromium.org

[commit-bot: I haz the power, 2014-07-31 01:19:10]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dnicoara@chromium.org/427853004/60001

[commit-bot: I haz the power, 2014-07-31 01:25:45]

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

[commit-bot: I haz the power, 2014-07-31 01:29:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-31 01:29:15]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  ios_rel_device_ninja on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  win8_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)

[dnicoara, 2014-07-31 14:31:01]

The CQ bit was checked by dnicoara@chromium.org

[commit-bot: I haz the power, 2014-07-31 14:32:28]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dnicoara@chromium.org/427853004/80001

[commit-bot: I haz the power, 2014-07-31 18:01:15]

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

[commit-bot: I haz the power, 2014-07-31 18:31:52]

Change committed as 286855