Move video encode accelerator IPC messages to GPU IO thread

media/gpu/media_foundation_video_encode_accelerator_win.h

 // Copyright 2016 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_GPU_MEDIA_FOUNDATION_VIDEO_ENCODE_ACCELERATOR_WIN_H_
 #define MEDIA_GPU_MEDIA_FOUNDATION_VIDEO_ENCODE_ACCELERATOR_WIN_H_
 
 #include <mfapi.h>
 #include <mfidl.h>
 #include <stdint.h>
@@ skipping 10 matching lines @@
 #include "base/threading/thread_checker.h"
 #include "base/win/scoped_comptr.h"
 #include "media/gpu/media_gpu_export.h"
 #include "media/video/video_encode_accelerator.h"
 
 namespace media {
 
 // Media Foundation implementation of the VideoEncodeAccelerator interface for
 // Windows.
 // This class saves the task runner on which it is constructed and returns
-// encoded data to the client using that same task runner. This class has
-// DCHECKs to makes sure that methods are called in sequence. It starts an
-// internal encoder thread on which VideoEncodeAccelerator implementation tasks
-// are posted.
+// encoded data to the client using that same task runner. If
+// TryToSetupEncodeOnSeparateThread() is called, it use=s the given

[sandersd (OOO until July 31), 2016/10/21 01:01:00]

Remove '='

[emircan, 2016/10/24 19:44:53]

Done.

+// |encode_task_runner| instead. This class has DCHECKs to makes sure that
+// methods are called in sequence. It starts an internal encoder thread on which
+// VideoEncodeAccelerator implementation tasks are posted.
 class MEDIA_GPU_EXPORT MediaFoundationVideoEncodeAccelerator
     : public VideoEncodeAccelerator {
  public:
   MediaFoundationVideoEncodeAccelerator();
 
   // VideoEncodeAccelerator implementation.
   VideoEncodeAccelerator::SupportedProfiles GetSupportedProfiles() override;
   bool Initialize(VideoPixelFormat input_format,
                   const gfx::Size& input_visible_size,
                   VideoCodecProfile output_profile,
                   uint32_t initial_bitrate,
                   Client* client) override;
   void Encode(const scoped_refptr<VideoFrame>& frame,
               bool force_keyframe) override;
   void UseOutputBitstreamBuffer(const BitstreamBuffer& buffer) override;
   void RequestEncodingParametersChange(uint32_t bitrate,
                                        uint32_t framerate) override;
   void Destroy() override;
+  bool TryToSetupEncodeOnSeparateThread(
+      const base::WeakPtr<Client>& encode_client,
+      const scoped_refptr<base::SingleThreadTaskRunner>& encode_task_runner)
+      override;
 
   // Preload dlls required for encoding.
   static void PreSandboxInitialization();
 
  protected:
   ~MediaFoundationVideoEncodeAccelerator() override;
 
  private:
   // Holds output buffers coming from the client ready to be filled.
   struct BitstreamBufferRef;
@@ skipping 60 matching lines @@
 
   base::win::ScopedComPtr<IMFSample> input_sample_;
   base::win::ScopedComPtr<IMFSample> output_sample_;
 
   // To expose client callbacks from VideoEncodeAccelerator.
   // NOTE: all calls to this object *MUST* be executed on |client_task_runner_|.
   base::WeakPtr<Client> client_;
   std::unique_ptr<base::WeakPtrFactory<Client>> client_ptr_factory_;
 
   // Our original calling task runner for the child thread.
-  const scoped_refptr<base::SequencedTaskRunner> client_task_runner_;
-  // Sequence checker to enforce that the methods of this object are called in
-  // sequence.
+  scoped_refptr<base::SequencedTaskRunner> client_task_runner_;
+
+  // Thread checker to enforce that this object created, initialized and
+  // destroyed on a specific thread. It is pinned on the thread where object is
+  // created.
+  base::ThreadChecker thread_checker_;
+
+  // Sequence checker to enforce that the encode tasks of this object are called
+  // in sequence.
   base::SequenceChecker sequence_checker_;
 
   // This thread services tasks posted from the VEA API entry points by the
   // GPU child thread and CompressionCallback() posted from device thread.
   base::Thread encoder_thread_;
   scoped_refptr<base::SingleThreadTaskRunner> encoder_thread_task_runner_;
 
   // Declared last to ensure that all weak pointers are invalidated before
   // other destructors run.
   base::WeakPtrFactory<MediaFoundationVideoEncodeAccelerator>
       encoder_task_weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(MediaFoundationVideoEncodeAccelerator);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_GPU_MEDIA_FOUNDATION_VIDEO_ENCODE_ACCELERATOR_WIN_H_