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

Unified Diff: media/video/omx_video_decode_engine.cc

Issue 3335014: Added FakeGlVideoDecodeEngine to exercise the IPC protocol for hardware video decoding (Closed)
Patch Set: compile man... Created 10 years, 3 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: media/video/omx_video_decode_engine.cc
diff --git a/media/video/omx_video_decode_engine.cc b/media/video/omx_video_decode_engine.cc
index b953cfacec738381283e40d3270fde22650b9e94..e1bcc7f92341ba57d82d0f7b77f6672a7fb8d54c 100644
--- a/media/video/omx_video_decode_engine.cc
+++ b/media/video/omx_video_decode_engine.cc
@@ -88,8 +88,8 @@ void OmxVideoDecodeEngine::Initialize(
message_loop_ = message_loop;
event_handler_ = event_handler;
- width_ = config.width_;
- height_ = config.height_;
+ width_ = config.width;
+ height_ = config.height;
// TODO(wjia): Find the right way to determine the codec type.
OmxConfigurator::MediaFormat input_format, output_format;
@@ -107,14 +107,14 @@ void OmxVideoDecodeEngine::Initialize(
VideoCodecInfo info;
// TODO(jiesun): ridiculous, we never fail initialization?
- info.success_ = true;
- info.provides_buffers_ = !uses_egl_image_;
- info.stream_info_.surface_type_ =
+ info.success = true;
+ info.provides_buffers = !uses_egl_image_;
+ info.stream_info.surface_type =
uses_egl_image_ ? VideoFrame::TYPE_GL_TEXTURE
: VideoFrame::TYPE_SYSTEM_MEMORY;
- info.stream_info_.surface_format_ = GetSurfaceFormat();
- info.stream_info_.surface_width_ = config.width_;
- info.stream_info_.surface_height_ = config.height_;
+ info.stream_info.surface_format = GetSurfaceFormat();
+ info.stream_info.surface_width = config.width;
+ info.stream_info.surface_height = config.height;
event_handler_->OnInitializeComplete(info);
}

Powered by Google App Engine
This is Rietveld 408576698