1079278 - indexedDB xpcshell tests inadvertently disabled on android

Status: RESOLVED FIXED

(Core :: Storage: IndexedDB, defect)

Description

[Andrew Halberstadt [:ahal]]

Bug 994190 renamed dom/indexedDB/test/unit/xpcshell.ini to dom/indexedDB/test/unit/xpcshell-shared.ini but unfortunately forgot to update the root android manifest in the process:
http://dxr.mozilla.org/mozilla-central/source/testing/xpcshell/xpcshell_android.ini

Horribly enough, the harness spit out a warning, but nothing that turned the job orange, so this went unnoticed. In bug 1066735 I'm working to remove the android and b2g specific xpcshell manifests so this doesn't happen again in the future.

Comment 1

[Andrew Halberstadt [:ahal]]

Attachment: Re-enable indexedDB xpcshell tests on Android

Hopefully nothing broke in the meantime:
https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=a2f300ef2832

Comment 2

[Andrew Halberstadt [:ahal]]

That was wishful thinking:
https://treeherder.mozilla.org/ui/logviewer.html#?job_id=2299119&repo=try

Ben, I'll leave it up to you whether you want to fix or disable this test.

Comment 3

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Let's just disable that one test, it's somehow related to the way we do packaging on android and I don't have time to look at it right now.

Comment 4

[Andrew Halberstadt [:ahal]]

Attachment: Re-enable indexedDB xpcshell tests on Android

Also disables test_globalObjects_other.js on android.

Comment 5

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Comment on attachment 8501904 [details] [diff] [review]
Re-enable indexedDB xpcshell tests on Android

Review of attachment 8501904 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!

Comment 6

[Andrew Halberstadt [:ahal]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/50a379edc315

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/50a379edc315