Closed
Bug 1234230
Opened 7 years ago
Closed 6 years ago
MediaRecorder video blob has blinking when recording both image and audio
Categories
(Core :: Audio/Video: Recording, defect, P1)
Tracking
()
RESOLVED
FIXED
mozilla47
People
(Reporter: ladybenko, Assigned: jesup)
References
Details
(Keywords: DevAdvocacy, regression)
Attachments
(1 file)
3.07 KB,
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
When using MediaRecorder to record a stream of both video and audio, the resulting video blob contains glitches (it's blinking). See: https://jsfiddle.net/56eefo2x/7/ When recording video only, (asking for just video in getUserMedia) this does not happen. Also, if the getUserMedia stream is piped to a videotag and we wait till the loadedmetadata event before starting the recording, the video blob is fine. I'm not sure if this is a bug, but it's somehow counter-intuitive that I don't need to wait for loadedmetadata when recording video only, but I have to when recording both audio and video.
Reporter | ||
Updated•7 years ago
|
Keywords: DevAdvocacy
Updated•7 years ago
|
Rank: 15
Priority: -- → P1
Comment 1•6 years ago
|
||
I wanted to report the same issue but I found this bug report first. The bug can also be seen here: https://simpl.info/mediarecorder/ In Firefox 43, everything worked fine. In Firefox 44, it doesn't work on some systems. Downgrading to Firefox 43 makes it work again. Rebooting, refreshing settings, uninstall and reinstall of FF 44, using a different user has no effect. I tried it on four systems running Windows 7. The two that failed have a Logitech C920 webcam. Here are sample recordings from before and after the upgrade to FF 44: https://hypermeeting.paldeploy.com/mturk/firefox43.webm https://hypermeeting.paldeploy.com/mturk/firefox44.webm
Updated•6 years ago
|
Keywords: regressionwindow-wanted
![]() |
||
Comment 2•6 years ago
|
||
There are 2 regression window, #1, flicker with very high rate(5-10Hz in my feeling) #2, slightly improved, but flicker with low rate(2-3 Hz in my feeling) #1 Regression window: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=0a5b448cd09ea78e7ef930c3910b488ac27efebc&tochange=19400ffcf4695ae195d79770c9e7db574b558e66 #1 Regressed by: Bug 953265 #2 Regression window: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=8d8eb725a4d977944fa4a660d5405fbcb26beb6b&tochange=cc01ca9f3dff6577aa0aaad1f3c68e0bdfa79760 #2 Regressed by: Bug 1218593
status-firefox44:
--- → affected
status-firefox45:
--- → affected
status-firefox47:
--- → affected
Flags: needinfo?(rjesup)
Flags: needinfo?(alwu)
Keywords: regressionwindow-wanted → regression
OS: Mac OS X → All
![]() |
||
Comment 3•6 years ago
|
||
Via local build: Last Good: aa7c3ce047ad First Bad: 0cf3e430dc22 Regressed by: 0cf3e430dc22 Randell Jesup — Bug 953265: Update webrtc/getUserMedia default audio capture rate to 32KHz r=padenot
Flags: needinfo?(padenot)
Assignee | ||
Comment 4•6 years ago
|
||
The proximate cause for the flashing was apparently null frames due to blocking. While I see them in both working and non-working tests, timings and durations are different and this appears to be timing sensitive. (For example I didn't see the regression at the 16->32KHz landing.) Note this patch may cause frames to be encoded with start times slightly earlier than they should be; correcting this as noted in the comment requires either delaying encoding by a frame, or passing in a timestamp instead of a duration. This should be handled in a followup bug, as we want this patch to uplift to 46. Nice side-effect - this may improve quality by avoiding re-encoding the same image multiple times. (Note: borrows heavily from OMXEncoder, which also implies we could remove some duplication in the future.) In local testing, the flashing is gone with this.
Attachment #8724756 -
Flags: review?(roc)
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → rjesup
Status: NEW → ASSIGNED
Assignee | ||
Updated•6 years ago
|
Attachment #8724756 -
Flags: review?(roc) → review+
Updated•6 years ago
|
Flags: needinfo?(alwu)
Comment 7•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/0ac27aa6a4d8
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Updated•6 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•