Closed
Bug 843469
Opened 13 years ago
Closed 13 years ago
Set WebRTC video defaults to 30 FPS and put prefs in all.js for easy about:config editing
Categories
(Core :: WebRTC: Audio/Video, defect)
Core
WebRTC: Audio/Video
Tracking
()
RESOLVED
DUPLICATE
of bug 842715
People
(Reporter: dmosedale, Unassigned)
References
Details
(Whiteboard: [getUserMedia][blocking-gum-][android-trunk-needed][android-webrtc+])
Attachments
(1 file)
|
2.93 KB,
patch
|
jesup
:
review-
|
Details | Diff | Splinter Review |
+++ This bug is a followon for Bug #842715 +++
After discussion with jesup and mreavy, this is two minor tweaks to the gUM prefs, initially for landing on alder, later for uplift to mozilla-central.
| Reporter | ||
Updated•13 years ago
|
Summary: Set FPS defaults to 30 and put prefs in all.js for easy about:config editing → Set WebRTC video defaults to 30 FPS and put prefs in all.js for easy about:config editing
| Reporter | ||
Comment 1•13 years ago
|
||
| Reporter | ||
Updated•13 years ago
|
Whiteboard: [getUserMedia][blocking-gum-] → [getUserMedia][blocking-gum-][android-trunk-needed]
| Reporter | ||
Comment 2•13 years ago
|
||
Landed on alder:
https://hg.mozilla.org/projects/alder/rev/761f25e7c9c6
| Reporter | ||
Updated•13 years ago
|
Attachment #716371 -
Attachment description: default prefs and set default fps to 30 → default prefs and set default fps to 30 (landed on alder)
| Reporter | ||
Updated•13 years ago
|
Whiteboard: [getUserMedia][blocking-gum-][android-trunk-needed] → [getUserMedia][blocking-gum-][android-trunk-needed][android-webrtc+]
Comment 3•13 years ago
|
||
Comment on attachment 716371 [details] [diff] [review]
default prefs and set default fps to 30 (landed on alder)
Review of attachment 716371 [details] [diff] [review]:
-----------------------------------------------------------------
and minfps is actually the one that's used for many things, not fps!
::: modules/libpref/src/init/all.js
@@ +178,5 @@
> pref("media.navigator.enabled", true);
> +pref("media.navigator.video.default_width", 640);
> +pref("media.navigator.video.default_height", 480);
> +pref("media.navigator.video.default_fps", 30);
> +pref("media.navigator.video.default_min", 10);
minfps....
Attachment #716371 -
Flags: review-
Updated•13 years ago
|
Blocks: android-webrtc
Comment 4•13 years ago
|
||
Is this already landed at this point? Is there any work left to do here?
Flags: needinfo?(rjesup)
Comment 5•13 years ago
|
||
Landed on m-i/m-c via bug 842715
Status: NEW → RESOLVED
Closed: 13 years ago
Flags: needinfo?(rjesup)
Resolution: --- → DUPLICATE
Updated•13 years ago
|
No longer blocks: android-webrtc
You need to log in
before you can comment on or make changes to this bug.
Description
•