Closed Bug 1302320 Opened 8 years ago Closed 8 years ago

add a test to confirm that video control show "noaudio" icon when video has no audio

Categories

(Toolkit :: Video/Audio Controls, defect)

Product:
Toolkit
Component:
Video/Audio Controls
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla53
Tracking Flags:
Tracking Status
firefox53 --- fixed

People

(Reporter: ralin, Assigned: ralin)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Bug 1302320 - add noaudio icon test for video control.
(deleted), text/x-review-board-request
jaws
: review+
Details
We should add a test to chek video UI when video has no audio. The test need to confirm that "noaudio" show up, in other hand, mute/unmute button and volume control are hidden.
Blocks: 1303958
Assignee: nobody → ralin
Status: NEW → ASSIGNED
temporarily hold from review for bug 1271765
Comment on attachment 8801023 [details] Bug 1302320 - add noaudio icon test for video control. https://reviewboard.mozilla.org/r/85832/#review84980 ::: toolkit/content/tests/widgets/mochitest.ini:7 (Diff revision 1) > skip-if = buildapp == 'mulet' || buildapp == 'b2g' > support-files = > audio.wav > audio.ogg > seek_with_sound.ogg > + video.ogg What's the size of the video file? Did you |git add| it? Can we reuse /mozilla-central/source/browser/base/content/test/general/video.ogg ?
The file is existed on repo alreay, I didn't do |git add| here. It looks like this file has never been used since 2008(first added).
Depends on: 1271765
Comment on attachment 8801023 [details] Bug 1302320 - add noaudio icon test for video control. https://reviewboard.mozilla.org/r/85832/#review94996
Attachment #8801023 - Flags: review?(jaws) → review+
Keywords: checkin-needed
Pushed by ihsiao@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/84fee6da47a6 add noaudio icon test for video control. r=jaws
Keywords: checkin-needed
sorry had to backout for test_videocontrols_video_noaudio.html assertion failure, https://treeherder.mozilla.org/logviewer.html#?job_id=7078899&repo=autoland#L2408
Flags: needinfo?(ralin)
Thank you Iris, I'll go fix it or maybe consider excluding this test from Android.
Flags: needinfo?(ralin)
Backout by ihsiao@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8ea8de32a32a Backed out changeset 84fee6da47a6 for test_videocontrols_video_noaudio.html assertion failure
Skipped this test on Android platform due to mobile video control doesn't care about `noaudio` state.
Keywords: checkin-needed
Pushed by cbook@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4e27de747d61 add noaudio icon test for video control. r=jaws
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/4e27de747d61
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: