Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(377)

Unified Diff: remoting/base/encoder_vp8.cc

Issue 7084013: Update VP8 encode options to speed up encoding for remoting (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: removed some comments Created 9 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/base/encoder_vp8.cc
diff --git a/remoting/base/encoder_vp8.cc b/remoting/base/encoder_vp8.cc
index 0bba179a97cd397ef3b6002163ea013053fcf0f2..c21c33d43e78340cf3654b30bfdcb4ed640459df 100644
--- a/remoting/base/encoder_vp8.cc
+++ b/remoting/base/encoder_vp8.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+B1;2305;0c// Copyright (c) 2011 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
@@ -99,14 +99,13 @@ bool EncoderVp8::Init(const gfx::Size& size) {
active_map_height_ = (size.height() + kMacroBlockSize - 1) / kMacroBlockSize;
active_map_.reset(new uint8[active_map_width_ * active_map_height_]);
- // TODO(hclam): Tune the parameters to better suit the application.
config.rc_target_bitrate = size.width() * size.height() *
config.rc_target_bitrate / config.g_w / config.g_h;
config.g_w = size.width();
config.g_h = size.height();
config.g_pass = VPX_RC_ONE_PASS;
- config.g_profile = 1;
- config.g_threads = 1;
+ config.g_profile = 2;
+ config.g_threads = 2;
config.rc_min_quantizer = 20;
config.rc_max_quantizer = 30;
config.g_timebase.num = 1;
@@ -114,6 +113,10 @@ bool EncoderVp8::Init(const gfx::Size& size) {
if (vpx_codec_enc_init(codec_.get(), algo, &config, 0))
return false;
+ if (vpx_codec_control(codec_.get(), VP8E_SET_CPUUSED, 16))
dmac 2011/05/28 00:16:33 Can we put some comments here as to why these valu
+ return false;
+ if (vpx_codec_control(codec_.get(), VP8E_SET_NOISE_SENSITIVITY, 0))
+ return false;
return true;
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698