| Index: media/gpu/mojo/service/gpu_jpeg_decode_accelerator.h
 | 
| diff --git a/media/gpu/ipc/service/gpu_jpeg_decode_accelerator.h b/media/gpu/mojo/service/gpu_jpeg_decode_accelerator.h
 | 
| similarity index 58%
 | 
| rename from media/gpu/ipc/service/gpu_jpeg_decode_accelerator.h
 | 
| rename to media/gpu/mojo/service/gpu_jpeg_decode_accelerator.h
 | 
| index 811ad6ec6ad12a51bf118ad72334d10c84767a82..2a6fa0f85bea993e4ee60e5413e2789ca94a7399 100644
 | 
| --- a/media/gpu/ipc/service/gpu_jpeg_decode_accelerator.h
 | 
| +++ b/media/gpu/mojo/service/gpu_jpeg_decode_accelerator.h
 | 
| @@ -11,20 +11,16 @@
 | 
|  
 | 
|  #include "base/macros.h"
 | 
|  #include "base/memory/ref_counted.h"
 | 
| -#include "base/memory/weak_ptr.h"
 | 
|  #include "base/sequence_checker.h"
 | 
| -#include "base/synchronization/waitable_event.h"
 | 
| -#include "ipc/ipc_listener.h"
 | 
| -#include "ipc/ipc_sender.h"
 | 
| -#include "media/gpu/mojo/jpeg_decoder.mojom.h"
 | 
| +#include "media/gpu/mojo/common/jpeg_decoder.mojom.h"
 | 
|  #include "media/video/jpeg_decode_accelerator.h"
 | 
|  
 | 
|  namespace base {
 | 
|  class SingleThreadTaskRunner;
 | 
|  }
 | 
|  
 | 
| -namespace gpu {
 | 
| -class FilteredSender;
 | 
| +namespace service_manager {
 | 
| +struct BindSourceInfo;
 | 
|  }
 | 
|  
 | 
|  namespace media {
 | 
| @@ -42,34 +38,26 @@ class GpuJpegDecodeAcceleratorFactoryProvider {
 | 
|    static std::vector<CreateAcceleratorCB> GetAcceleratorFactories();
 | 
|  };
 | 
|  
 | 
| -// TODO(c.padhi): Move GpuJpegDecodeAccelerator to media/gpu/mojo, see
 | 
| -// http://crbug.com/699255.
 | 
| -class GpuJpegDecodeAccelerator
 | 
| -    : public IPC::Sender,
 | 
| -      public mojom::GpuJpegDecodeAccelerator,
 | 
| -      public base::SupportsWeakPtr<GpuJpegDecodeAccelerator> {
 | 
| +class GpuJpegDecodeAccelerator : public mojom::GpuJpegDecodeAccelerator,
 | 
| +                                 public JpegDecodeAccelerator::Client {
 | 
|   public:
 | 
| -  // |channel| must outlive this object.
 | 
| -  // This convenience constructor internally calls
 | 
| -  // GpuJpegDecodeAcceleratorFactoryProvider::GetAcceleratorFactories() to
 | 
| -  // fill |accelerator_factory_functions_|.
 | 
| -  GpuJpegDecodeAccelerator(
 | 
| -      gpu::FilteredSender* channel,
 | 
| -      scoped_refptr<base::SingleThreadTaskRunner> io_task_runner);
 | 
| +  static void Create(scoped_refptr<base::SingleThreadTaskRunner> io_task_runner,
 | 
| +                     const service_manager::BindSourceInfo& source_info,
 | 
| +                     mojom::GpuJpegDecodeAcceleratorRequest request);
 | 
|  
 | 
| -  // |channel| must outlive this object.
 | 
| -  GpuJpegDecodeAccelerator(
 | 
| -      gpu::FilteredSender* channel,
 | 
| -      scoped_refptr<base::SingleThreadTaskRunner> io_task_runner,
 | 
| -      std::vector<GpuJpegDecodeAcceleratorFactoryProvider::CreateAcceleratorCB>
 | 
| -          accelerator_factory_functions);
 | 
|    ~GpuJpegDecodeAccelerator() override;
 | 
|  
 | 
| -  void AddClient(int32_t route_id, base::Callback<void(bool)> response);
 | 
| +  // JpegDecodeAccelerator::Client implementation.
 | 
| +  void VideoFrameReady(int32_t buffer_id) override;
 | 
| +  void NotifyError(int32_t buffer_id,
 | 
| +                   JpegDecodeAccelerator::Error error) override;
 | 
|  
 | 
| -  void NotifyDecodeStatus(int32_t route_id,
 | 
| -                          int32_t bitstream_buffer_id,
 | 
| -                          JpegDecodeAccelerator::Error error);
 | 
| + private:
 | 
| +  // This constructor internally calls
 | 
| +  // GpuJpegDecodeAcceleratorFactoryProvider::GetAcceleratorFactories() to
 | 
| +  // fill |accelerator_factory_functions_|.
 | 
| +  GpuJpegDecodeAccelerator(
 | 
| +      scoped_refptr<base::SingleThreadTaskRunner> io_task_runner);
 | 
|  
 | 
|    // mojom::GpuJpegDecodeAccelerator implementation.
 | 
|    void Initialize(InitializeCallback callback) override;
 | 
| @@ -80,31 +68,24 @@ class GpuJpegDecodeAccelerator
 | 
|                DecodeCallback callback) override;
 | 
|    void Uninitialize() override;
 | 
|  
 | 
| -  // Function to delegate sending to actual sender.
 | 
| -  bool Send(IPC::Message* message) override;
 | 
| -
 | 
| - private:
 | 
| -  class Client;
 | 
| -  class MessageFilter;
 | 
| -
 | 
| -  void ClientRemoved();
 | 
| +  void DecodeOnIOThread(mojom::BitstreamBufferPtr input_buffer,
 | 
| +                        const gfx::Size& coded_size,
 | 
| +                        mojo::ScopedSharedBufferHandle output_handle,
 | 
| +                        uint32_t output_buffer_size);
 | 
| +  void NotifyDecodeStatus(int32_t bitstream_buffer_id, mojom::Error error);
 | 
|  
 | 
|    std::vector<GpuJpegDecodeAcceleratorFactoryProvider::CreateAcceleratorCB>
 | 
|        accelerator_factory_functions_;
 | 
|  
 | 
| -  gpu::FilteredSender* channel_;
 | 
| -
 | 
| -  // The message filter to run JpegDecodeAccelerator::Decode on IO thread.
 | 
| -  scoped_refptr<MessageFilter> filter_;
 | 
| -
 | 
|    // GPU child task runner.
 | 
|    scoped_refptr<base::SingleThreadTaskRunner> child_task_runner_;
 | 
|  
 | 
|    // GPU IO task runner.
 | 
|    scoped_refptr<base::SingleThreadTaskRunner> io_task_runner_;
 | 
|  
 | 
| -  // Number of clients added to |filter_|.
 | 
| -  int client_number_;
 | 
| +  DecodeCallback decode_cb_;
 | 
| +
 | 
| +  std::unique_ptr<JpegDecodeAccelerator> accelerator_;
 | 
|  
 | 
|    SEQUENCE_CHECKER(sequence_checker_);
 | 
|  
 | 
| 
 |