VideoTrackRecorder: Adding inner class H264Encoder

content/renderer/media/video_track_recorder.h

 // Copyright 2015 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_VIDEO_TRACK_RECORDER_H_
 #define CONTENT_RENDERER_MEDIA_VIDEO_TRACK_RECORDER_H_
 
 #include <memory>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/threading/thread_checker.h"
+#include "content/public/common/features.h"

[hbos_chromium, 2016/04/20 20:28:42]

(FYI this is a generated header file so any target with sources including this
need to depend on content/content.gyp:common_features (gyp) and
//content/public/common:features (GN) or you could get flakey compilation in
some cases. Though this target already depends on it so it's good.)

[mcasas, 2016/04/20 23:20:34]

Acknowledged.

 #include "content/public/renderer/media_stream_video_sink.h"
 #include "third_party/WebKit/public/platform/WebMediaStreamTrack.h"
 
 namespace media {
 class VideoFrame;
 }  // namespace media
 
 namespace content {
 
 // VideoTrackRecorder is a MediaStreamVideoSink that encodes the video frames
 // received from a Stream Video Track. This class is constructed and used on a
 // single thread, namely the main Render thread. This mirrors the other
 // MediaStreamVideo* classes that are constructed/configured on Main Render
 // thread but that pass frames on Render IO thread. It has an internal Encoder
 // with its own threading subtleties, see the implementation file.
 class CONTENT_EXPORT VideoTrackRecorder
     : NON_EXPORTED_BASE(public MediaStreamVideoSink) {
  public:
   enum class CodecId {
     VP8,
     VP9,
+#if BUILDFLAG(RTC_USE_H264)
+    H264,
+#endif
   };
   class Encoder;
 
   using OnEncodedVideoCB =
       base::Callback<void(const scoped_refptr<media::VideoFrame>& video_frame,
                           std::unique_ptr<std::string> encoded_data,
                           base::TimeTicks capture_timestamp,
                           bool is_key_frame)>;
 
   VideoTrackRecorder(CodecId codec,
@@ skipping 10 matching lines @@
  private:
   friend class VideoTrackRecorderTest;
 
   // Used to check that we are destroyed on the same thread we were created.
   base::ThreadChecker main_render_thread_checker_;
 
   // We need to hold on to the Blink track to remove ourselves on dtor.
   blink::WebMediaStreamTrack track_;
 
   // Inner class to encode using whichever codec is configured.
-  const scoped_refptr<Encoder> encoder_;
+  scoped_refptr<Encoder> encoder_;
 
   DISALLOW_COPY_AND_ASSIGN(VideoTrackRecorder);
 };
 
 }  // namespace content
 #endif  // CONTENT_RENDERER_MEDIA_VIDEO_TRACK_RECORDER_H_