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

Unified Diff: remoting/codec/video_encoder_vpx.cc

Issue 26921005: Add VP9 decode support to the remoting client. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Disable VP9 correctly in hosts. Created 7 years, 2 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
Index: remoting/codec/video_encoder_vpx.cc
diff --git a/remoting/codec/video_encoder_vpx.cc b/remoting/codec/video_encoder_vpx.cc
index c9194024cf7253ce3ccc381bb9041bd1345f5585..9259ea706f38ccf6943d4a19cb70e861c209e9fc 100644
--- a/remoting/codec/video_encoder_vpx.cc
+++ b/remoting/codec/video_encoder_vpx.cc
@@ -197,6 +197,10 @@ bool VideoEncoderVpx::Initialize(const webrtc::DesktopSize& size) {
image_->d_h = size.height();
image_->h = size.height();
+ // libvpx should derive this from|fmt| but currently has a bug.
+ image_->x_chroma_shift = 1;
+ image_->y_chroma_shift = 1;
Jamie 2013/10/21 18:28:20 Does this bug only affect VP9? If not, then this s
Wez 2013/10/22 01:13:01 We only need to set this due to a bug in libvpx. I
+
// Initialize active map.
active_map_width_ = (image_->w + kMacroBlockSize - 1) / kMacroBlockSize;
active_map_height_ = (image_->h + kMacroBlockSize - 1) / kMacroBlockSize;

Powered by Google App Engine
This is Rietveld 408576698