1619340 - Explicitly disable individual gtests that fail on Android, instead of entire files

Status: RESOLVED FIXED

(Core :: Panning and Zooming, task, P3)

Description

[Kartikaya Gupta (email:[e-mail])]

Spinoff from https://bugzilla.mozilla.org/show_bug.cgi?id=1545234#c14. This just ifdef's out individual failing tests instead of skipping the entire file on Android. Subsequent bugs/patches can fix and re-enable the individual tests.

Comment 1

[Kartikaya Gupta (email:[e-mail])]

https://treeherder.mozilla.org/#/jobs?repo=try&group_state=expanded&revision=53758fa5308800779448d01124c61350e2a1f477

Comment 2

[Kartikaya Gupta (email:[e-mail])]

https://treeherder.mozilla.org/#/jobs?repo=try&group_state=expanded&revision=187b989dcb48ac4bbe97ab9217f4d32d76e591ed is better. Some more tests are failing (with a crash on debug builds) than were some months ago when I last tried this. So landing earlier could have caught some regressions :(

Comment 3

[Kartikaya Gupta (email:[e-mail])]

Attachment: Bug 1619340 - ifdef out the individual gtests failing on Android. r?botond

Instead of skipping entire files, ifdef out the individual tests that are
failing. This allows us to audit them one at a time to re-enable, while
prevent regressions in tests in those files that are already passing.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/53e21d1350a5
ifdef out the individual gtests failing on Android. r=botond

Comment 5

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/53e21d1350a5