634696 - enable tp4, tp4_nochrome, twinopen and tgfx for android tegra's

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Mike Taylor [:bear]]

from jmaher in IRC:

tp4 and tp4_nochrome as well as twinopen and tgfx for android/tegra

Comment 1

[Joel Maher ( :jmaher ) (UTC -8) (PTO back normal Nov 17)]

with the latest fennecmark, tzoom is good.  I have had trouble on and off with tgfx, it seems to be crashing on android about halfway through the test.  everything else here runs good on my local setup.

Comment 2

[Mike Taylor [:bear]]

Attachment: enable remaining talos tests for android

Comment 3

[Aki Sasaki (not active)]

Comment on attachment 514994 [details] [diff] [review]
enable remaining talos tests for android

>@@ -290,17 +294,17 @@ PLATFORM_UNITTEST_VARS = {
>             },
>         },
>         'android': {
>             'builds_before_reboot':  1,
>             'enable_opt_unittests': False,
>             'enable_debug_unittests': False,
>             'download_symbols': False,
>             'tegra_android': {
>-                'opt_unittest_suites': [],
>+                'opt_unittest_suites': removeSuite('mochitest-a11y', UNITTEST_SUITES['opt_unittest_suites'][:]),
>                 'debug_unittest_suites': [],
>             },
>         },
> }

r=me if you remove this change... this is cruft, correct?

Comment 4

[Mike Taylor [:bear]]

Comment on attachment 514994 [details] [diff] [review]
enable remaining talos tests for android

committed changeset 3674:bde3ecc22f23

Comment 5

[Mike Taylor [:bear]]

Attachment: helps if you add the new tests to the default var loop at end

Comment 6

[Aki Sasaki (not active)]

Comment on attachment 515153 [details] [diff] [review]
helps if you add the new tests to the default var loop at end

That fixes it.

Comment 7

[Mike Taylor [:bear]]

Comment on attachment 515153 [details] [diff] [review]
helps if you add the new tests to the default var loop at end

committed changeset 3676:1fcd01047db3

Comment 8

[Joel Maher ( :jmaher ) (UTC -8) (PTO back normal Nov 17)]

my understanding is we have everything EXCEPT tgfx up and running for android/tegra.

Comment 9

[Aki Sasaki (not active)]

Wondering whether we should enable tgfx in light of bug 493050 and bug 589396.
But if you get that running, we can enable it.

Comment 10

[Aki Sasaki (not active)]

Per bug 638698, we're disabling tgfx on mobile devices.
We're done here.