Fix gyp failure on linux when use_ozone=1 is set

Fix gyp failure on linux when use_ozone=1 is set

BUG=489264
TEST=build/gyp_chromium -Duse_ozone=1

Committed: https://crrev.com/84251bbd423965f9db89331761185b5630147215
Cr-Commit-Position: refs/heads/master@{#330701}

[babu, 2015-05-18 19:33:40]

sudarsana.nagineni@intel.com changed reviewers:
+ dpranke@chromium.org

[Dirk Pranke, 2015-05-18 20:44:15]

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

[Dirk Pranke, 2015-05-18 20:44:16]

https://codereview.chromium.org/1139973005/diff/1/build/gn_migration.gypi
File build/gn_migration.gypi (right):

https://codereview.chromium.org/1139973005/diff/1/build/gn_migration.gypi#new...
build/gn_migration.gypi:243: ['remoting==1 and chromeos==0 and use_x11==1', {
This seems to be the right change to at least match what we do in
remoting/remoting_enable.gypi, but this doesn't actually solve the bug (which is
a gyp bug), right?

[babu, 2015-05-19 11:10:53]

https://codereview.chromium.org/1139973005/diff/1/build/gn_migration.gypi
File build/gn_migration.gypi (right):

https://codereview.chromium.org/1139973005/diff/1/build/gn_migration.gypi#new...
build/gn_migration.gypi:243: ['remoting==1 and chromeos==0 and use_x11==1', {
On 2015/05/18 20:44:16, Dirk Pranke wrote:
> This seems to be the right change to at least match what we do in
> remoting/remoting_enable.gypi, but this doesn't actually solve the bug (which
is
> a gyp bug), right?

Thanks for reviewing, Dirk!

This fixes the build error for me. My understanding is that the
gn_migration.gypi is adding remoting_me2me_* targets, but those targets are not
enabled in remoting/remoting_host.gypi when use_x11=0 and that is causing build
failure in the Linux builds. I might be wrong here though.

This patch fixes the issue by not trying to include those targets in
gn_migration.gypi when use_x11=0.

[Dirk Pranke, 2015-05-19 19:54:47]

https://codereview.chromium.org/1139973005/diff/1/build/gn_migration.gypi
File build/gn_migration.gypi (right):

https://codereview.chromium.org/1139973005/diff/1/build/gn_migration.gypi#new...
build/gn_migration.gypi:243: ['remoting==1 and chromeos==0 and use_x11==1', {
Oh, I'm sorry, I totally misread this and thought you were modifying a GN file
:(.

lgtm.

[babu, 2015-05-20 04:09:05]

The CQ bit was checked by sudarsana.nagineni@intel.com

[commit-bot: I haz the power, 2015-05-20 04:09:11]

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

[commit-bot: I haz the power, 2015-05-20 05:07:32]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-05-20 05:08:30]

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