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

Unified Diff: media/tools/player_wtl/wtl_renderer.cc

Issue 7461016: Replace VideoDecoder::media_format() with significantly simpler width()/height() methods. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src
Patch Set: Created 9 years, 5 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/tools/player_wtl/wtl_renderer.cc
diff --git a/media/tools/player_wtl/wtl_renderer.cc b/media/tools/player_wtl/wtl_renderer.cc
index 04a932c86d42a52afe5cb31da5288821bec60a40..baec88e5735fc4ae41c9055cb3bedb6e82c313cc 100644
--- a/media/tools/player_wtl/wtl_renderer.cc
+++ b/media/tools/player_wtl/wtl_renderer.cc
@@ -12,10 +12,9 @@ WtlVideoRenderer::WtlVideoRenderer(WtlVideoWindow* window)
WtlVideoRenderer::~WtlVideoRenderer() {}
-// static
-bool WtlVideoRenderer::IsMediaFormatSupported(
- const media::MediaFormat& media_format) {
- return ParseMediaFormat(media_format, NULL, NULL, NULL);
+bool WtlVideoRenderer::OnInitialize(media::VideoDecoder* decoder) {
+ window_->SetSize(decoder->width(), decoder->height());
+ return true;
}
void WtlVideoRenderer::OnStop(media::FilterCallback* callback) {
@@ -25,11 +24,6 @@ void WtlVideoRenderer::OnStop(media::FilterCallback* callback) {
}
}
-bool WtlVideoRenderer::OnInitialize(media::VideoDecoder* decoder) {
- window_->SetSize(width(), height());
- return true;
-}
-
void WtlVideoRenderer::OnFrameAvailable() {
window_->Invalidate();
}

Powered by Google App Engine
This is Rietveld 408576698