Add histogram stats for AV sync stream offset:

webrtc/video/vie_sync_module.h

 /*
  *  Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
 // ViESyncModule is responsible for synchronization audio and video for a given
 // VoE and ViE channel couple.
 
 #ifndef WEBRTC_VIDEO_VIE_SYNC_MODULE_H_
 #define WEBRTC_VIDEO_VIE_SYNC_MODULE_H_
 
 #include "webrtc/base/criticalsection.h"
 #include "webrtc/base/scoped_ptr.h"
 #include "webrtc/modules/include/module.h"
 #include "webrtc/system_wrappers/include/tick_util.h"
 #include "webrtc/video/stream_synchronization.h"
 #include "webrtc/voice_engine/include/voe_video_sync.h"
 
 namespace webrtc {
 
+class Clock;
 class RtpRtcp;
 class VideoCodingModule;
+class VideoFrame;
 class ViEChannel;
 class VoEVideoSync;
 
 class ViESyncModule : public Module {
  public:
   explicit ViESyncModule(VideoCodingModule* vcm);
   ~ViESyncModule();
 
   void ConfigureSync(int voe_channel_id,
                      VoEVideoSync* voe_sync_interface,
                      RtpRtcp* video_rtcp_module,
                      RtpReceiver* video_receiver);
 
   // Implements Module.
   int64_t TimeUntilNextProcess() override;
   void Process() override;
 
+  // Returns the absolute value of the sync offset between the video |frame| and
+  // the current played out audio frame (or -1 on error).

[pbos-webrtc, 2016/03/04 14:34:27]

-1 if not configured/in use?

[åsapersson, 2016/03/09 15:44:35]

Done.

+  int64_t GetStreamSyncOffsetInMs(const VideoFrame& frame);

[stefan-webrtc, 2016/03/04 14:32:17]

Would it be reasonable to request a unittest here too?

[åsapersson, 2016/03/09 15:44:35]

I will look into adding unittest for this class. Maybe in a separate CL?

[stefan-webrtc, 2016/03/09 15:59:30]

Sure no problem.

+
  private:
   rtc::CriticalSection data_cs_;
   VideoCodingModule* const vcm_;
+  Clock* const clock_;
   RtpReceiver* video_receiver_;
   RtpRtcp* video_rtp_rtcp_;
   int voe_channel_id_;
   VoEVideoSync* voe_sync_interface_;
   TickTime last_sync_time_;
   rtc::scoped_ptr<StreamSynchronization> sync_;
   StreamSynchronization::Measurements audio_measurement_;
   StreamSynchronization::Measurements video_measurement_;
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_VIDEO_VIE_SYNC_MODULE_H_