Expose Ivf logging through the native API

webrtc/video/video_receive_stream.h

Index: webrtc/video/video_receive_stream.h
diff --git a/webrtc/video/video_receive_stream.h b/webrtc/video/video_receive_stream.h
index 3cffb4b38585e83f8300c9b649ac7372e05d1e1f..f13406c51b24f4d62373f6258c0113c5f4b04dc3 100644
--- a/webrtc/video/video_receive_stream.h
+++ b/webrtc/video/video_receive_stream.h
@@ -84,6 +84,12 @@ class VideoReceiveStream : public webrtc::VideoReceiveStream,
   // Implements KeyFrameRequestSender.
   void RequestKeyFrame() override;
 
+  // Takes ownership of the file, is responsible for closing it later.
+  // Calling this method will close and finalize any current log.
+  // Giving rtc::kInvalidPlatformFileValue disables logging.
+  void SetLogFile(
+      rtc::PlatformFile file = rtc::kInvalidPlatformFileValue) override;

[sprang_webrtc, 2016/09/04 14:48:48]

I think this needs a more descriptive name.
Something like EnableEncodedFrameRecording() ?

+
  private:
   static bool DecodeThreadFunction(void* ptr);
   void Decode();
@@ -105,6 +111,7 @@ class VideoReceiveStream : public webrtc::VideoReceiveStream,
   std::unique_ptr<VideoStreamDecoder> video_stream_decoder_;
   RtpStreamsSynchronizer rtp_stream_sync_;
 
+  rtc::CriticalSection ivf_writer_crit_;

[sprang_webrtc, 2016/09/04 14:48:48]

I personally prefer lock as name, rather than crit, but up to you.

   std::unique_ptr<IvfFileWriter> ivf_writer_;

[sprang_webrtc, 2016/09/04 14:48:48]

add GUARDED_BY

 };
 }  // namespace internal