android: Don't run failing FontTest.DeriveFont; run gfx_unittests on bots.

android: run gfx_unittests on bots.

Also disable a few tests that don't pass on Android.

BUG=662559, 642010, 662556, 662558, 662561
Committed: https://crrev.com/c10e8e86d03fb5308bc00d1d468662e6eff59268
Cr-Commit-Position: refs/heads/master@{#431157}

[Nico, 2016-11-04 19:27:28]

The CQ bit was checked by thakis@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-04 19:28:00]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Nico, 2016-11-04 19:29:06]

thakis@chromium.org changed reviewers:
+ karandeepb@chromium.org, stip@chromium.org

[Nico, 2016-11-04 19:29:06]

karandeepb: Test disablement (see bug from some discussion)
stip: All the json copypasta (do we really need to repeat
"3ff24775a900b675866fbcacf2a8f98a18b2a16a" 145 times in that file?)

[commit-bot: I haz the power, 2016-11-04 20:34:16]

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

[commit-bot: I haz the power, 2016-11-04 20:34:17]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[karandeepb, 2016-11-07 00:00:23]

https://codereview.chromium.org/2483493002/diff/1/ui/gfx/font_unittest.cc
File ui/gfx/font_unittest.cc (right):

https://codereview.chromium.org/2483493002/diff/1/ui/gfx/font_unittest.cc#new...
ui/gfx/font_unittest.cc:165: #if defined(TOOLKIT_VIEWS)
Think this should be disabled explicitly on Android like the other tests in the
file. There isn't anything Views specific here.

[Nico, 2016-11-07 00:04:26]

https://codereview.chromium.org/2483493002/diff/1/ui/gfx/font_unittest.cc
File ui/gfx/font_unittest.cc (right):

https://codereview.chromium.org/2483493002/diff/1/ui/gfx/font_unittest.cc#new...
ui/gfx/font_unittest.cc:165: #if defined(TOOLKIT_VIEWS)
On 2016/11/07 00:00:22, karandeepb wrote:
> Think this should be disabled explicitly on Android like the other tests in
the
> file. There isn't anything Views specific here.

aelias's comment on the bug suggested otherwise, did you see that? If you
disagree with it, can you comment there?

[karandeepb, 2016-11-07 00:06:04]

On 2016/11/07 00:04:26, Nico (mostly afk Wed-Fri) wrote:
> https://codereview.chromium.org/2483493002/diff/1/ui/gfx/font_unittest.cc
> File ui/gfx/font_unittest.cc (right):
> 
>
https://codereview.chromium.org/2483493002/diff/1/ui/gfx/font_unittest.cc#new...
> ui/gfx/font_unittest.cc:165: #if defined(TOOLKIT_VIEWS)
> On 2016/11/07 00:00:22, karandeepb wrote:
> > Think this should be disabled explicitly on Android like the other tests in
> the
> > file. There isn't anything Views specific here.
> 
> aelias's comment on the bug suggested otherwise, did you see that? If you
> disagree with it, can you comment there?

Yeah saw that. Sure, will do.

[Nico, 2016-11-09 20:28:56]

The CQ bit was checked by thakis@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-09 20:29:38]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Nico, 2016-11-09 20:29:48]

Description was changed from

==========
android: Don't run failing FontTest.DeriveFont; run gfx_unittests on bots.

BUG=642010
==========

to

==========
android: run gfx_unittests on bots.

Also disable a few tests that don't pass on Android.

BUG=662559,642010,662556,662558,662561
==========

[Nico, 2016-11-09 20:30:58]

The CQ bit was checked by thakis@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-09 20:31:19]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Nico, 2016-11-09 20:33:44]

thakis@chromium.org changed reviewers:
+ danakj@chromium.org, enne@chromium.org

[Nico, 2016-11-09 20:33:45]

karandeepb: Done, please also review 	ui/gfx/platform_font_android.cc
stip: ping!
danakj, enne: Please look at the files where I added TODOs for you (enne:
transform_unittest.cc, danakj: vector3d_unittest, and quad_unittest)

[danakj, 2016-11-09 20:45:42]

LGTM

[ghost stip (do not use), 2016-11-09 20:46:01]

lgtm

[commit-bot: I haz the power, 2016-11-09 21:42:26]

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

[commit-bot: I haz the power, 2016-11-09 21:42:27]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[Nico, 2016-11-09 21:45:45]

The CQ bit was checked by thakis@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-09 21:46:23]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-09 23:37:12]

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

[commit-bot: I haz the power, 2016-11-09 23:37:13]

Dry run: This issue passed the CQ dry run.

[karandeepb, 2016-11-09 23:53:10]

LGTM

[Nico, 2016-11-10 02:54:34]

The CQ bit was checked by thakis@chromium.org

[Nico, 2016-11-10 02:54:35]

The patchset sent to the CQ was uploaded after l-g-t-m from danakj@chromium.org,
stip@chromium.org
Link to the patchset: https://codereview.chromium.org/2483493002/#ps60001
(title: "include build_config.h more")

[commit-bot: I haz the power, 2016-11-10 02:54:51]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-10 03:03:44]

Description was changed from

==========
android: run gfx_unittests on bots.

Also disable a few tests that don't pass on Android.

BUG=662559,642010,662556,662558,662561
==========

to

==========
android: run gfx_unittests on bots.

Also disable a few tests that don't pass on Android.

BUG=662559,642010,662556,662558,662561
==========

[commit-bot: I haz the power, 2016-11-10 03:03:45]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-11-10 03:08:40]

Description was changed from

==========
android: run gfx_unittests on bots.

Also disable a few tests that don't pass on Android.

BUG=662559,642010,662556,662558,662561
==========

to

==========
android: run gfx_unittests on bots.

Also disable a few tests that don't pass on Android.

BUG=662559,642010,662556,662558,662561

Committed: https://crrev.com/c10e8e86d03fb5308bc00d1d468662e6eff59268
Cr-Commit-Position: refs/heads/master@{#431157}
==========

[commit-bot: I haz the power, 2016-11-10 03:08:41]

Patchset 4 (id:??) landed as
https://crrev.com/c10e8e86d03fb5308bc00d1d468662e6eff59268
Cr-Commit-Position: refs/heads/master@{#431157}