Bug 1330676 - Use VBR in VP8TrackEncoder. r=bechen,derf
authorAndreas Pehrson <pehrsons@gmail.com>
Thu, 12 Jan 2017 17:13:13 +0100
changeset 374536 00f3061cb568898110c41a27297b14c2f48370fd
parent 374535 37d35f87e0fe226e6294aa7ef0c34a0d49a7fe19
child 374537 fd18ab48f5c3cbce6b9d6f26f0df4b0d436278a2
push id6996
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 20:48:21 +0000
treeherdermozilla-beta@d89512dab048 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbechen, derf
bugs1330676
milestone53.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1330676 - Use VBR in VP8TrackEncoder. r=bechen,derf MozReview-Commit-ID: AvezDCrOIuZ
dom/media/encoder/VP8TrackEncoder.cpp
--- a/dom/media/encoder/VP8TrackEncoder.cpp
+++ b/dom/media/encoder/VP8TrackEncoder.cpp
@@ -124,17 +124,17 @@ VP8TrackEncoder::Init(int32_t aWidth, in
   } else if (mFrameWidth * mFrameHeight > 640 * 480 && number_of_cores >= 3) {
     config.g_threads = 2; // 2 threads for qHD/HD.
   } else {
     config.g_threads = 1; // 1 thread for VGA or less
   }
 
   // rate control settings
   config.rc_dropframe_thresh = 0;
-  config.rc_end_usage = VPX_CBR;
+  config.rc_end_usage = VPX_VBR;
   config.g_pass = VPX_RC_ONE_PASS;
   // ffmpeg doesn't currently support streams that use resize.
   // Therefore, for safety, we should turn it off until it does.
   config.rc_resize_allowed = 0;
   config.rc_undershoot_pct = 100;
   config.rc_overshoot_pct = 15;
   config.rc_buf_initial_sz = 500;
   config.rc_buf_optimal_sz = 600;