Remove media::VideoDecodeContext as it has no implementation and is unused.

media/video/video_decode_engine.h

Index: media/video/video_decode_engine.h
diff --git a/media/video/video_decode_engine.h b/media/video/video_decode_engine.h
index cd1f9860402e4eafb716c2103d88d86abbfab13f..cb08f2ab0e4982267afb70d3581db32f5493752d 100644
--- a/media/video/video_decode_engine.h
+++ b/media/video/video_decode_engine.h
@@ -12,13 +12,9 @@
 #include "media/base/video_frame.h"
 #include "ui/gfx/size.h"
 
-class MessageLoop;
-
 namespace media {
 
 class Buffer;
-class VideoDecodeContext;
-
 struct PipelineStatistics;
 
 class MEDIA_EXPORT VideoDecodeEngine {
@@ -67,17 +63,9 @@ class MEDIA_EXPORT VideoDecodeEngine {
 
   // Initialize the engine with specified configuration.
   //
-  // |decode_context| is used for allocation of VideoFrame.
-  // It is important that |decode_context| is called only on |message_loop|.
-  //
-  // TODO(hclam): Currently refactoring code to use VideoDecodeContext so
-  // |context| may be NULL in some cases.
-  //
   // Engine should call EventHandler::OnInitializeDone() whether the
   // initialization operation finished successfully or not.
-  virtual void Initialize(MessageLoop* message_loop,

[Ami GONE FROM CHROMIUM, 2011/10/28 16:47:31]

What is up w/ dropping message_loop_ from the engine Initialize() method?  

[scherkus (not reviewing), 2011/10/28 17:00:38]

it's not even used by FFVDE haha

[Ami GONE FROM CHROMIUM, 2011/10/28 17:23:17]

Wow.

-                          EventHandler* event_handler,
-                          VideoDecodeContext* context,
+  virtual void Initialize(EventHandler* event_handler,
                           const VideoDecoderConfig& config) = 0;
 
   // Uninitialize the engine. Engine should destroy all resources and call