Disable failing TextEliderTest.* and FontListTest.*

Disable failing TextEliderTest.* and FontListTest.*

This CL:
- disables the remaining TextEliderTest.* on Android
- disables more failing FontListTest.* on Android

BUG=338784, 489354
Committed: https://crrev.com/85fd1d9526b59acbac9cdb04cb0adb2af1be0b55
Cr-Commit-Position: refs/heads/master@{#330749}

[pkotwicz, 2015-05-18 17:57:11]

pkotwicz@chromium.org changed reviewers:
+ jbudorick@chromium.org, msw@chromium.org

[pkotwicz, 2015-05-18 17:57:11]

jbudorick@ can you please take a look at the changes in
	build/android/pylib/gtest/filter/gfx_unittests_disabled

msw@ can you please take a look at the chanegs in ui/gfx

[pkotwicz, 2015-05-18 17:58:30]

Patchset #1 (id:1) has been deleted

[jbudorick, 2015-05-18 18:10:05]

I don't agree with moving the text elider test disabling from the code to the
build files. I think it makes it much harder to see that those tests are
disabled on Android.

https://codereview.chromium.org/1145473006/diff/20001/build/android/pylib/gte...
File build/android/pylib/gtest/filter/gfx_unittests_disabled (right):

https://codereview.chromium.org/1145473006/diff/20001/build/android/pylib/gte...
build/android/pylib/gtest/filter/gfx_unittests_disabled:9:
FontListTest.Fonts_DeriveWithHeightUpperBound
I would prefer to have these done in code rather than adding to this file.
(Ideally, I'd like to get rid of these files and have all of the disables done
in code.)

[pkotwicz, 2015-05-18 19:14:44]

jbudorick@ can you please take a look? I have moved the disabling into the test
files

[jbudorick, 2015-05-18 19:17:56]

Thanks for moving all of the FontListTest disables out of there!

https://codereview.chromium.org/1145473006/diff/40001/ui/gfx/font_list_unitte...
File ui/gfx/font_list_unittest.cc (right):

https://codereview.chromium.org/1145473006/diff/40001/ui/gfx/font_list_unitte...
ui/gfx/font_list_unittest.cc:93: #if !defined(OS_ANDROID)
I think this should be done via the MAYBE_FooTest pattern that the other tests
use. That way at least compiles the tests on Android.

[pkotwicz, 2015-05-18 20:59:06]

jbudorick@ can you please take another look?

https://codereview.chromium.org/1145473006/diff/40001/ui/gfx/font_list_unitte...
File ui/gfx/font_list_unittest.cc (right):

https://codereview.chromium.org/1145473006/diff/40001/ui/gfx/font_list_unitte...
ui/gfx/font_list_unittest.cc:93: #if !defined(OS_ANDROID)
On 2015/05/18 19:17:56, jbudorick wrote:
> I think this should be done via the MAYBE_FooTest pattern that the other tests
> use. That way at least compiles the tests on Android.

Done.

[jbudorick, 2015-05-19 21:53:59]

build/android/ lgtm

[jbudorick, 2015-05-19 21:54:09]

On 2015/05/19 21:53:59, jbudorick wrote:
> build/android/ lgtm

and thanks for the cleanup!

[msw, 2015-05-19 21:57:57]

lgtm

[pkotwicz, 2015-05-20 15:40:29]

The CQ bit was checked by pkotwicz@chromium.org

[commit-bot: I haz the power, 2015-05-20 15:40:45]

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

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

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2015-05-20 16:37:27]

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