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

RESOLVED FIXED in Firefox 54

Status

()

Core
Audio/Video: Playback
RESOLVED FIXED
11 months ago
9 months ago

People

(Reporter: jya, Assigned: alwu)

Tracking

({regression})

unspecified
mozilla55
Unspecified
Android
regression
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox51 unaffected, firefox52 unaffected, firefox-esr52 unaffected, firefox53 unaffected, firefox54 fixed, firefox55 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

(Reporter)

Description

11 months ago
Permafailing following bug 1319987 and bug 1336358

Updated

11 months ago
status-firefox51: --- → unaffected
status-firefox52: --- → unaffected
status-firefox53: --- → unaffected
status-firefox54: --- → affected

Comment 1

11 months ago
jya: next steps here?  getting near to 54 going to aurora.  Who should look at this?
Flags: needinfo?(jyavenard)
(Reporter)

Comment 2

11 months ago
(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.
Flags: needinfo?(jyavenard)

Updated

11 months ago
Summary: Re-enable video-replay-after-audio-end.html crashtest → Re-enable video-replay-after-audio-end.html crashtest (Android emulator)

Comment 3

11 months ago
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
Flags: needinfo?(jyavenard)

Comment 4

10 months ago
We need to check this, and disable the test or do something, if it's still happening
Flags: needinfo?(rjesup)

Comment 5

10 months ago
(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?
Alastor, 
Please check this on autophone. 
Thanks.
Flags: needinfo?(rjesup)
Flags: needinfo?(jyavenard)
Flags: needinfo?(alwu)
(Assignee)

Comment 7

9 months ago
Created attachment 8858764 [details]
Pass log in Nexus 5, Android 6.0.1

The test can pass on my Nexus5 with Android 6.0.1.
Flags: needinfo?(alwu)
Alastor,
Any update?
Flags: needinfo?(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
From the try server result, it seems we can enable this test again.
Flags: needinfo?(alwu)
Comment hidden (mozreview-request)
(Reporter)

Comment 13

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

https://reviewboard.mozilla.org/r/133238/#review136142
Attachment #8861284 - Flags: review?(jyavenard) → review+

Comment 14

9 months ago
Pushed by alwu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/66311a113b58
Re-enable video-replay-after-audio-end.html crashtest. r=jya

Comment 15

9 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/66311a113b58
Status: NEW → RESOLVED
Last Resolved: 9 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Were you going to check 54 as well?
Assignee: nobody → alwu
status-firefox-esr52: --- → unaffected
Flags: needinfo?(alwu)
Could I push the changed to the branch 54? because I don't sure whether this test can run on 54.
Flags: needinfo?(alwu) → needinfo?(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
Flags: needinfo?(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.
Whiteboard: [checkin-needed-beta]

Comment 20

9 months ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-beta/rev/fdf45ba25f3d
status-firefox54: affected → fixed
Whiteboard: [checkin-needed-beta]
You need to log in before you can comment on or make changes to this bug.