1339449 - Re-enable video-replay-after-audio-end.html crashtest (Android emulator)

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect)

Description

[Jean-Yves Avenard [:jya]]

Permafailing following bug 1319987 and bug 1336358

Comment 1

[Randell Jesup [:jesup] (needinfo me)]

jya: next steps here?  getting near to 54 going to aurora.  Who should look at this?

Comment 2

[Jean-Yves Avenard [:jya]]

(In reply to Randell Jesup [:jesup] from comment #1)
> jya: next steps here?  getting near to 54 going to aurora.  Who should look
> at this?

while treeherder is using supra-slow simulator to run mochitest, I don't think there's much we can do about it.

Comment 3

[Randell Jesup [:jesup] (needinfo me)]

Does it fail on autophone runs?  Those are real HW.

If it doesn't, we add it to the bucket of "too-slow emulator" bugs

Comment 4

[Randell Jesup [:jesup] (needinfo me)]

We need to check this, and disable the test or do something, if it's still happening

Comment 5

[Nathan Froyd [:froydnj]]

(In reply to Randell Jesup [:jesup] from comment #3)
> Does it fail on autophone runs?  Those are real HW.
> 
> If it doesn't, we add it to the bucket of "too-slow emulator" bugs

Who's the right person to talk to about whether this fails on autophone?  Do those actually show up on treeherder?

Comment 6

[Blake Wu [:bwu][:blakewu]]

Alastor, 
Please check this on autophone. 
Thanks.

Comment 7

[Alastor Wu [:alwu]]

Attachment: Pass log in Nexus 5, Android 6.0.1

The test can pass on my Nexus5 with Android 6.0.1.

Comment 8

[Alastor Wu [:alwu]]

Try server link with running this test.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=efe944c076dbc9b242e0eae9d775f5c4f47757bb

Comment 9

[Blake Wu [:bwu][:blakewu]]

Alastor,
Any update?

Comment 10

[Alastor Wu [:alwu]]

Let's run the test again to see whether we can re-enable this test.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=3ecf9f7cda32b535803b5a290df12ba5ba19ed57

Comment 11

[Alastor Wu [:alwu]]

From the try server result, it seems we can enable this test again.

Comment 12

[Alastor Wu [:alwu]]

Attachment: Bug 1339449 - Re-enable video-replay-after-audio-end.html crashtest.

Review commit: https://reviewboard.mozilla.org/r/133238/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/133238/

Comment 13

[Jean-Yves Avenard [:jya]]

Comment on attachment 8861284 [details]
Bug 1339449 - Re-enable video-replay-after-audio-end.html crashtest.

https://reviewboard.mozilla.org/r/133238/#review136142

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/66311a113b58
Re-enable video-replay-after-audio-end.html crashtest. r=jya

Comment 15

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

https://hg.mozilla.org/mozilla-central/rev/66311a113b58

Comment 16

[Ryan VanderMeulen [:RyanVM]]

Were you going to check 54 as well?

Comment 17

[Alastor Wu [:alwu]]

Could I push the changed to the branch 54? because I don't sure whether this test can run on 54.

Comment 18

[Ryan VanderMeulen [:RyanVM]]

You can push it to Try on top of a Beta tree, yes. I went ahead and did so to cut down on lag here :)
https://treeherder.mozilla.org/#/jobs?repo=try&revision=5045bcde46e243fff302ac1e5f58664f4f10aee0

Comment 19

[Ryan VanderMeulen [:RyanVM]]

Green on Try, including lots of retriggers. This is test-only, so I'm marking it to ride-along the next time we're pushing there anyway.

Comment 20

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-beta/rev/fdf45ba25f3d