render creation require media format have mime type and width/height

media/filters/omx_video_decoder.cc

 // Copyright (c) 2010 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.
 
 #include "media/filters/omx_video_decoder.h"
 
 #include "base/callback.h"
 #include "base/waitable_event.h"
 #include "media/base/factory.h"
 #include "media/base/filter_host.h"
+#include "media/base/limits.h"
 #include "media/ffmpeg/ffmpeg_common.h"
 #include "media/filters/ffmpeg_interfaces.h"
 #include "media/filters/omx_video_decode_engine.h"
 
 namespace media {
 
 // static
 FilterFactory* OmxVideoDecoder::CreateFactory() {
   return new FilterFactoryImpl1<OmxVideoDecoder, OmxVideoDecodeEngine*>(
       new OmxVideoDecodeEngine());
@@ skipping 54 matching lines @@
   // TODO(ajwong): This is a total hack. Make async.
   base::WaitableEvent event(false, false);
   omx_engine_->Stop(NewCallback(&event, &base::WaitableEvent::Signal));
   event.Wait();
 }
 
 void OmxVideoDecoder::DoInitialize(DemuxerStream* demuxer_stream,
                                    FilterCallback* callback) {
   DCHECK_EQ(message_loop(), MessageLoop::current());
 
+  // Get the AVStream by querying for the provider interface.
+  AVStreamProvider* av_stream_provider;
+  if (!demuxer_stream->QueryInterface(&av_stream_provider)) {
+    return;
+  }
+  AVStream* av_stream = av_stream_provider->GetAVStream();
+
+  width_ = av_stream->codec->width;
+  height_ = av_stream->codec->height;
+  if (width_ > Limits::kMaxDimension ||
+      height_ > Limits::kMaxDimension ||
+      (width_ * height_) > Limits::kMaxCanvas) {
+    return;
+  }
+
   // Sets the output format.
   if (supports_egl_image_) {
     media_format_.SetAsString(MediaFormat::kMimeType,
                               mime_type::kUncompressedVideoEglImage);
   }
+  else {
+    media_format_.SetAsString(MediaFormat::kMimeType,
+                              mime_type::kUncompressedVideo);
+  }
+
+  media_format_.SetAsInteger(MediaFormat::kWidth, width_);
+  media_format_.SetAsInteger(MediaFormat::kHeight, height_);
 
   // Savs the demuxer stream.
   demuxer_stream_ = demuxer_stream;
 
-  // Get the AVStream by querying for the provider interface.
-  AVStreamProvider* av_stream_provider;
-  if (!demuxer_stream->QueryInterface(&av_stream_provider))
-    return;
-  AVStream* av_stream = av_stream_provider->GetAVStream();
-
   // Initialize the decode engine.
   omx_engine_->Initialize(
       message_loop(),
       av_stream,
       NewCallback(this, &OmxVideoDecoder::EmptyBufferCallback),
       NewCallback(this, &OmxVideoDecoder::FillBufferCallback),
       NewRunnableMethod(this, &OmxVideoDecoder::InitCompleteTask, callback));
 }
 
 void OmxVideoDecoder::FillBufferCallback(scoped_refptr<VideoFrame> frame) {
@@ skipping 43 matching lines @@
   // TODO(hclam): Query this number from |omx_engine_|.
   const int kDemuxPackets = 2;
   DCHECK(demuxer_stream_);
   for (int i = 0; i < kDemuxPackets; ++i) {
     demuxer_stream_->Read(
         NewCallback(this, &OmxVideoDecoder::DemuxCompleteTask));
   }
 }
 
 }  // namespace media