code cleanup for media code for video renerers.

add more generic surface format and surface type signalling from video decoder to video renderer. if we want to allocate buffer in renderer,  we had to know these kind of information.
remove redundant ParseMediaFormat and duplicated width,height, egl_image_supports in derived class of VideoRenderBase.

Review URL: http://codereview.chromium.org/2829009

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50558 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 13 deletions

diff --git a/chrome/renderer/media/ipc_video_renderer.cc b/chrome/renderer/media/ipc_video_renderer.cc
index aab8f60..843869c 100644
--- a/chrome/renderer/media/ipc_video_renderer.cc
+++ b/chrome/renderer/media/ipc_video_renderer.cc
@@ -37,24 +37,14 @@ media::FilterFactory* IPCVideoRenderer::CreateFactory(
 // static
 bool IPCVideoRenderer::IsMediaFormatSupported(
     const media::MediaFormat& media_format) {
-  int width = 0;
-  int height = 0;
-  bool uses_egl_image = false;
-  return ParseMediaFormat(media_format, &width, &height, &uses_egl_image);
+  return ParseMediaFormat(media_format, NULL, NULL, NULL, NULL);
 }
 
 bool IPCVideoRenderer::OnInitialize(media::VideoDecoder* decoder) {
-  int width = 0;
-  int height = 0;
-  bool uses_egl_image = false;
-  if (!ParseMediaFormat(decoder->media_format(), &width, &height,
-                        &uses_egl_image))
-    return false;
-
-  video_size_.SetSize(width, height);
+  video_size_.SetSize(width(), height());
 
   // TODO(scherkus): we're assuming YV12 here.
-  size_t size = (width * height) + ((width * height) >> 1);
+  size_t size = (width() * height()) + ((width() * height()) >> 1);
   uint32 epoch = static_cast<uint32>(reinterpret_cast<size_t>(this));
   transport_dib_.reset(TransportDIB::Create(size, epoch));
   CHECK(transport_dib_.get());