1223261 - [Testcase] Audio should be playback correctly when open the pref "dom.audiochannel.mutedByDefault"

Status: RESOLVED FIXED

(Firefox OS Graveyard :: AudioChannel, defect)

Description

[Alastor Wu [:alwu]]

In b2g, the channel is muted by default, and then unmuted by the system app.
We need to simulate this case to make sure the basic audio playback can work successfully.

Comment 1

[Alastor Wu [:alwu]]

Attachment: MozReview Request: Bug 1223261 - Add the audio playback test for the case muted-by-default. r=baku.

Bug 1223261 - Add the audio playback test for the case muted-by-default.

Comment 2

[Alastor Wu [:alwu]]

Comment on attachment 8685246 [details]
MozReview Request: Bug 1223261 - Add the audio playback test for the case muted-by-default. r=baku.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/24769/diff/1-2/

Comment 3

[Andrea Marchesini [:baku]]

Comment on attachment 8685246 [details]
MozReview Request: Bug 1223261 - Add the audio playback test for the case muted-by-default. r=baku.

https://reviewboard.mozilla.org/r/24769/#review22299

Comment 4

[Alastor Wu [:alwu]]

Try-server result.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=15ca020f6e0e

Comment 5

[Alastor Wu [:alwu]]

Trying to fix the test case fail, new try-server result.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=436295364bf9

Comment 6

[Alastor Wu [:alwu]]

Still has test case fail, new try-server result.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=bbd68aa63b49

Comment 7

[Alastor Wu [:alwu]]

Comment on attachment 8685246 [details]
MozReview Request: Bug 1223261 - Add the audio playback test for the case muted-by-default. r=baku.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/24769/diff/2-3/

Comment 8

[Andrea Marchesini [:baku]]

Comment on attachment 8685246 [details]
MozReview Request: Bug 1223261 - Add the audio playback test for the case muted-by-default. r=baku.

https://reviewboard.mozilla.org/r/24769/#review22539

Comment 9

[Pulsebot]

https://hg.mozilla.org/integration/b2g-inbound/rev/da1bcceb5f57

Comment 10

[Carsten Book [:Tomcat]]

sorry backed out for test failurs like https://treeherder.mozilla.org/logviewer.html#?job_id=3354320&repo=b2g-inbound

Comment 11

[Pulsebot]

Backout:
https://hg.mozilla.org/integration/b2g-inbound/rev/4cad242946dd

Comment 12

[Alastor Wu [:alwu]]

Hi, Carsten,
I think this test faliure is resulted by other changeset, not my patch.
This patch doesn't include any Gecko changing.
It just changes the content of "test_browserElement_inproc(oop)_AudioChannel", so it shouldn't break any other test.

Comment 13

[Alastor Wu [:alwu]]

Hi, Wes,
Could you help me check-in this patch?
Thanks!

Comment 14

[Alastor Wu [:alwu]]

Maybe find the reason for timeout, update patch later.

Comment 15

[Alastor Wu [:alwu]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=9cce7d751f0e

Comment 16

[Alastor Wu [:alwu]]

Comment on attachment 8685246 [details]
MozReview Request: Bug 1223261 - Add the audio playback test for the case muted-by-default. r=baku.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/24769/diff/3-4/

Comment 17

[Pulsebot]

https://hg.mozilla.org/integration/b2g-inbound/rev/9ebcec50fefa

Comment 18

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

https://hg.mozilla.org/mozilla-central/rev/9ebcec50fefa

Comment 19

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Backed out in:
https://hg.mozilla.org/mozilla-central/rev/cc9c6cd756cb
for an unacceptable level of intermittent orange.  See dependent bugs above or in commit message.

Comment 20

[Alastor Wu [:alwu]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=59e5f4d65aa0

Comment 21

[Alastor Wu [:alwu]]

Comment on attachment 8685246 [details]
MozReview Request: Bug 1223261 - Add the audio playback test for the case muted-by-default. r=baku.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/24769/diff/4-5/

Comment 22

[Carsten Book [:Tomcat]]

Hi Alastor,

applying 0514fe364d43
patching file dom/browser-element/mochitest/browserElement_AudioChannel.js
Hunk #3 FAILED at 167
1 out of 3 hunks FAILED -- saving rejects to file dom/browser-element/mochitest/browserElement_AudioChannel.js.rej
patch failed to apply
abort: fix up the merge and run hg transplant --continue

could you take a look, thanks!

Comment 23

[Alastor Wu [:alwu]]

Comment on attachment 8685246 [details]
MozReview Request: Bug 1223261 - Add the audio playback test for the case muted-by-default. r=baku.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/24769/diff/5-6/

Comment 24

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/68ff75086870

Comment 25

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/68ff75086870