[Android] Set enable_remoting=true by default

[Android] Set enable_remoting=true by default

We set enable_remoting=true in the command line on the official builders. Set
enable_remoting=true as the default on Chromium as well.

BUG=544298
Committed: https://crrev.com/9fdef29316fae0cf8ba6c7258b434432d40eb175
Cr-Commit-Position: refs/heads/master@{#371604}

[pkotwicz, 2016-01-21 20:06:39]

Description was changed from

==========
[Android] Set enable_remoting=true by default

We set enable_remoting=true in the command line on the official builders. Do
the same in chromium.

BUG=544298
==========

to

==========
[Android] Set enable_remoting=true by default

We set enable_remoting=true in the command line on the official builders. Set
enable_remoting=true as the default on Chromium as well.

BUG=544298
==========

[pkotwicz, 2016-01-26 16:04:25]

pkotwicz@chromium.org changed reviewers:
+ lambroslambrou@chromium.org

[pkotwicz, 2016-01-26 16:04:25]

lambroslambrou@ can you please take a look?

[Lambros, 2016-01-26 18:00:40]

lgtm

[pkotwicz, 2016-01-26 18:02:33]

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

[pkotwicz, 2016-01-26 18:02:34]

dpranke@ for OWNERS

[Dirk Pranke, 2016-01-26 18:20:26]

lgtm

[pkotwicz, 2016-01-26 20:11:25]

The CQ bit was checked by pkotwicz@chromium.org

[commit-bot: I haz the power, 2016-01-26 20:12:51]

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

[commit-bot: I haz the power, 2016-01-26 20:37:57]

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

[commit-bot: I haz the power, 2016-01-26 20:37:58]

Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)

[pkotwicz, 2016-01-26 21:14:37]

The CQ bit was checked by pkotwicz@chromium.org

[commit-bot: I haz the power, 2016-01-26 21:15:56]

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

[commit-bot: I haz the power, 2016-01-26 21:38:38]

Description was changed from

==========
[Android] Set enable_remoting=true by default

We set enable_remoting=true in the command line on the official builders. Set
enable_remoting=true as the default on Chromium as well.

BUG=544298
==========

to

==========
[Android] Set enable_remoting=true by default

We set enable_remoting=true in the command line on the official builders. Set
enable_remoting=true as the default on Chromium as well.

BUG=544298
==========

[commit-bot: I haz the power, 2016-01-26 21:38:40]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-26 21:39:50]

Description was changed from

==========
[Android] Set enable_remoting=true by default

We set enable_remoting=true in the command line on the official builders. Set
enable_remoting=true as the default on Chromium as well.

BUG=544298
==========

to

==========
[Android] Set enable_remoting=true by default

We set enable_remoting=true in the command line on the official builders. Set
enable_remoting=true as the default on Chromium as well.

BUG=544298

Committed: https://crrev.com/9fdef29316fae0cf8ba6c7258b434432d40eb175
Cr-Commit-Position: refs/heads/master@{#371604}
==========

[commit-bot: I haz the power, 2016-01-26 21:39:50]

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

[pkotwicz, 2016-01-27 01:49:42]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1634353003/ by pkotwicz@chromium.org.

The reason for reverting is: This CL broke some of the internal builders which
are building "all".

remoting_cardboard_extract_native_lib should be behind an target_arch=="arm" if
statement.

[Lambros, 2016-01-27 01:58:26]

On 2016/01/27 01:49:42, pkotwicz wrote:
> A revert of this CL (patchset #1 id:1) has been created in
> https://codereview.chromium.org/1634353003/ by mailto:pkotwicz@chromium.org.
> 
> The reason for reverting is: This CL broke some of the internal builders which
> are building "all".
> 
> remoting_cardboard_extract_native_lib should be behind an target_arch=="arm"
if
> statement.

I see. In GYP (remoting/remoting_android.gypi), the target is defined but it's
only depended upon by remoting_apk if target_arch == "arm".

I guess the target *itself* needs to be behind the same conditional?