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

media/video/video_decode_engine.h

 // Copyright (c) 2011 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.
 
 #ifndef MEDIA_VIDEO_VIDEO_DECODE_ENGINE_H_
 #define MEDIA_VIDEO_VIDEO_DECODE_ENGINE_H_
 
 #include "base/callback.h"
 #include "base/memory/scoped_ptr.h"
 #include "media/base/media_export.h"
 #include "media/base/video_decoder_config.h"
 #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 {
  public:
   struct MEDIA_EXPORT EventHandler {
    public:
     virtual ~EventHandler() {}
     virtual void OnInitializeComplete(bool success) = 0;
     virtual void OnUninitializeComplete() = 0;
     virtual void OnFlushComplete() = 0;
@@ skipping 28 matching lines @@
     // method is called to return the video frame object to the owner.
     // The content of the video frame may be invalid.
     virtual void ConsumeVideoFrame(scoped_refptr<VideoFrame> frame,
                                    const PipelineStatistics& statistics) = 0;
   };
 
   virtual ~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
   // EventHandler::OnUninitializeComplete().
   virtual void Uninitialize() = 0;
 
   // Flush the engine. Engine should return all the buffers to owner ( which
   // could be itself. ) then call EventHandler::OnFlushDone().
   virtual void Flush() = 0;
 
@@ skipping 20 matching lines @@
   // is called by the user to return the video frame object.
   //
   // In response to this method call, ConsumeVideoFrame() is called with a
   // video frame object containing decoded video content.
   virtual void ProduceVideoFrame(scoped_refptr<VideoFrame> frame) = 0;
 };
 
 }  // namespace media
 
 #endif  // MEDIA_VIDEO_VIDEO_DECODE_ENGINE_H_