Make component_unittests run the same tests on GYP and GN on Android

Make component_unittests run the same tests on GYP and GN on Android

BUG=510485
R=skyostil
TBR=sdefresne

Committed: https://crrev.com/5927208e019e3d94ae008075f9165eef9ffe33c0
Cr-Commit-Position: refs/heads/master@{#356911}

[pkotwicz, 2015-10-26 23:56:07]

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

[pkotwicz, 2015-10-26 23:56:07]

blundell@ can you please take a look?

I did not add error_reporter_unittest.cc to the GN build yet even though it is
run on the GYP build because it fails on GN on Desktop Linux

[blundell, 2015-10-27 08:30:27]

blundell@chromium.org changed reviewers:
+ sdefresne@chromium.org
- blundell@chromium.org

[blundell, 2015-10-27 08:30:28]

blundell -> sdefresne

Sylvain is a better reviewer than me for this, as he's involved with bringing up
GN on iOS and can say definitively what should/shouldn't be built in GN on iOS
right now.

[Sami, 2015-10-27 09:58:14]

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

[Sami, 2015-10-27 09:58:14]

components/scheduler lgtm, thanks for the fix!

[pkotwicz, 2015-10-28 20:01:04]

sdefresne@ Ping!

[sdefresne, 2015-10-29 10:42:18]

l-g-t-m but please rebase to get iOS fixes in components/sessions/BUILD.gn

https://codereview.chromium.org/1418543006/diff/20001/components/sessions/BUI...
File components/sessions/BUILD.gn (left):

https://codereview.chromium.org/1418543006/diff/20001/components/sessions/BUI...
components/sessions/BUILD.gn:153: if (!is_ios && !is_android) {
This file has already been updated to work on iOS by
https://codereview.chromium.org/1417743003.
Could you ensure you keep it working there when rebasing?

[pkotwicz, 2015-10-29 18:16:53]

Description was changed from

==========
Make component_unittests run the same tests on GYP and GN on Android

BUG=510485
==========

to

==========
Make component_unittests run the same tests on GYP and GN on Android

BUG=510485
R=skyostil
TBR=sdefresne
==========

[pkotwicz, 2015-10-29 18:16:56]

The CQ bit was checked by pkotwicz@chromium.org

[pkotwicz, 2015-10-29 18:16:56]

The patchset sent to the CQ was uploaded after l-g-t-m from
skyostil@chromium.org
Link to the patchset: https://codereview.chromium.org/1418543006/#ps40001
(title: " ")

[commit-bot: I haz the power, 2015-10-29 18:17:10]

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

[commit-bot: I haz the power, 2015-10-29 19:29:21]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-10-29 19:30:03]

Patchset 3 (id:??) landed as
https://crrev.com/5927208e019e3d94ae008075f9165eef9ffe33c0
Cr-Commit-Position: refs/heads/master@{#356911}

[sdefresne, 2015-10-30 09:40:41]

lgtm