Make RTCVideoEncoder usable from non-postable thread.

content/renderer/media/rtc_video_encoder.h

 // Copyright 2013 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 CONTENT_RENDERER_MEDIA_RTC_VIDEO_ENCODER_H_
 #define CONTENT_RENDERER_MEDIA_RTC_VIDEO_ENCODER_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 14 matching lines @@
 class GpuVideoAcceleratorFactories;
 }  // namespace media
 
 namespace content {
 
 // RTCVideoEncoder uses a media::VideoEncodeAccelerator to implement a
 // webrtc::VideoEncoder class for WebRTC.  Internally, VEA methods are
 // trampolined to a private RTCVideoEncoder::Impl instance.  The Impl class runs
 // on the worker thread queried from the |gpu_factories_|, which is presently
 // the media thread.  RTCVideoEncoder is sychronized by webrtc::VideoSender.
-// webrtc::VideoEncoder methods do not run concurrently. RTCVideoEncoder is run
-// and destroyed on the thread it is constructed on, which is presently the
-// libjingle worker thread. Encode is run on ViECaptureThread. SetRates and
-// SetChannelParameters are run on ProcessThread or the libjingle worker thread.
-// Callbacks from the Impl due to its VEA::Client notifications are posted back
-// to RTCVideoEncoder on the libjingle worker thread.
+// webrtc::VideoEncoder methods do not run concurrently. RtcVideoEncoder needs
+// to synchronize RegisterEncodeCompleteCallback and encode complete callback.
 class CONTENT_EXPORT RTCVideoEncoder
     : NON_EXPORTED_BASE(public webrtc::VideoEncoder) {
  public:
   RTCVideoEncoder(webrtc::VideoCodecType type,
                   media::GpuVideoAcceleratorFactories* gpu_factories);
   ~RTCVideoEncoder() override;
 
   // webrtc::VideoEncoder implementation.  Tasks are posted to |impl_| using the
   // appropriate VEA methods.
   int32_t InitEncode(const webrtc::VideoCodec* codec_settings,
                      int32_t number_of_cores,
                      size_t max_payload_size) override;
   int32_t Encode(
       const webrtc::VideoFrame& input_image,
       const webrtc::CodecSpecificInfo* codec_specific_info,
       const std::vector<webrtc::FrameType>* frame_types) override;
   int32_t RegisterEncodeCompleteCallback(
       webrtc::EncodedImageCallback* callback) override;
   int32_t Release() override;
   int32_t SetChannelParameters(uint32_t packet_loss, int64_t rtt) override;
   int32_t SetRates(uint32_t new_bit_rate, uint32_t frame_rate) override;
 
  private:
   class Impl;
   friend class RTCVideoEncoder::Impl;
 
-  // Return an encoded output buffer to WebRTC.
-  void ReturnEncodedImage(std::unique_ptr<webrtc::EncodedImage> image,
-                          int32_t bitstream_buffer_id,
-                          uint16_t picture_id);
-
-  void NotifyError(int32_t error);
-
   void RecordInitEncodeUMA(int32_t init_retval,
                            media::VideoCodecProfile profile);
 
-  base::ThreadChecker thread_checker_;
-
-  // The video codec type, as reported to WebRTC.
-  const webrtc::VideoCodecType video_codec_type_;
-
   // Factory for creating VEAs, shared memory buffers, etc.
   media::GpuVideoAcceleratorFactories* gpu_factories_;
 
   // Task runner that the video accelerator runs on.
   const scoped_refptr<base::SingleThreadTaskRunner> gpu_task_runner_;
 
-  // webrtc::VideoEncoder encode complete callback.
-  webrtc::EncodedImageCallback* encoded_image_callback_;
-
   // The RTCVideoEncoder::Impl that does all the work.
   scoped_refptr<Impl> impl_;
 
-  // We cannot immediately return error conditions to the WebRTC user of this
-  // class, as there is no error callback in the webrtc::VideoEncoder interface.
-  // Instead, we cache an error status here and return it the next time an
-  // interface entry point is called.
-  int32_t impl_status_;
-
-  // Weak pointer factory for posting back VEA::Client notifications to
-  // RTCVideoEncoder.
-  // NOTE: Weak pointers must be invalidated before all other member variables.
-  base::WeakPtrFactory<RTCVideoEncoder> weak_factory_;
-
   DISALLOW_COPY_AND_ASSIGN(RTCVideoEncoder);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_MEDIA_RTC_VIDEO_ENCODER_H_