1039582 - Investigate disabling parallel tests in content/media/test

Status: RESOLVED DUPLICATE of bug 1036992

(Core :: Audio/Video, defect)

Description

[Syd Polk :sydpolk]

Currently, test_seek.html is highly intermittent, so much so that it has been disabled for b2g. (see https://bugzilla.mozilla.org/show_bug.cgi?id=981153). This investigation will enable it for b2g and Android, but not run tests in parallel. On my MacBook Pro, running with PARALLEL_TESTS = 2 takes 11 minutes; with PARALLEL_TESTS = 1 takes 17 minutes.

Comment 1

[cmtalbert]

Got syd's patch in email. Pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=939398bcc18f

Also, Syd, you need to configure your editors to use linux only line endings.

Please attach your patches here.

Comment 2

[Syd Polk :sydpolk]

Attachment: 1039582.patch

Comment 3

[Syd Polk :sydpolk]

Attachment: This patch re-enables test_seek.html on B2G/Android AND sets PARALLEL_TASKS to 1

Comment 4

[JW Wang [:jwwang] [:jw_wang]]

Please refer to [1]. We should set PARALLEL_TESTS to 1 only in test_seek.html when it is B2G or Android in order not to slow down tests on other platforms.

[1] http://hg.mozilla.org/mozilla-central/file/4024d8019701/content/media/test/manifest.js#l258

Comment 5

[Syd Polk :sydpolk]

jwwang: Your point is certainly well-taken. Right now we are doing investigation. If indeed we decide to proceed in this direction, we will be doing what you suggest. Thanks!

Comment 6

[cmtalbert]

Syd's second patch on try:  https://tbpl.mozilla.org/?tree=Try&rev=9601b6bffd11

Comment 8

[Syd Polk :sydpolk]

This patch caused too many failures as many tests depend on PARALLEL_TESTS being > 1. Investigation subsumed by 1036992.