Vanquish the remnants of media::MessageLoopFactory.

media/filters/gpu_video_decoder.h

 // Copyright (c) 2012 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_FILTERS_GPU_VIDEO_DECODER_H_
 #define MEDIA_FILTERS_GPU_VIDEO_DECODER_H_
 
 #include <list>
 #include <map>
 #include <utility>
@@ skipping 37 matching lines @@
     virtual void DeleteTexture(uint32 texture_id) = 0;
 
     // Read pixels from a native texture and store into |*pixels| as RGBA.
     virtual void ReadPixels(uint32 texture_id, uint32 texture_target,
                             const gfx::Size& size, void* pixels) = 0;
 
     // Allocate & return a shared memory segment.  Caller is responsible for
     // Close()ing the returned pointer.
     virtual base::SharedMemory* CreateSharedMemory(size_t size) = 0;
 
+    // Returns the message loop the VideoDecodeAccelerator runs on.
+    virtual scoped_refptr<base::MessageLoopProxy> GetMessageLoop() = 0;

[scherkus (not reviewing), 2012/12/07 23:54:11]

this is because we now create GpuVideoDecoder in webkit/ as opposed to content/
and can't access the proper loop

alternatively, we can create the thread + GpuVideoDecoder in content/ and
plumbing them into WMPI() (it better reflects the end goal of passing a
MessageLoopProxy of the shared media thread into WMPI) -- it gets a bit hairy
though with OverrideWMPI()

acowell: any thoughts / preferences?

+
    protected:
     friend class base::RefCountedThreadSafe<Factories>;
     virtual ~Factories();
   };
 
-  GpuVideoDecoder(const scoped_refptr<base::MessageLoopProxy>& gvd_loop_proxy,
-                  const scoped_refptr<base::MessageLoopProxy>& vda_loop_proxy,
+  GpuVideoDecoder(const scoped_refptr<base::MessageLoopProxy>& message_loop,
                   const scoped_refptr<Factories>& factories);
 
   // VideoDecoder implementation.
   virtual void Initialize(const scoped_refptr<DemuxerStream>& stream,
                           const PipelineStatusCB& status_cb,
                           const StatisticsCB& statistics_cb) OVERRIDE;
   virtual void Read(const ReadCB& read_cb) OVERRIDE;
   virtual void Reset(const base::Closure& closure) OVERRIDE;
   virtual void Stop(const base::Closure& closure) OVERRIDE;
   virtual bool HasAlpha() const OVERRIDE;
@@ skipping 146 matching lines @@
 
   // Indicates decoding error occurred.
   bool error_occured_;
 
   DISALLOW_COPY_AND_ASSIGN(GpuVideoDecoder);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_FILTERS_GPU_VIDEO_DECODER_H_