Make cricket::VideoCapturer implement VideoSourceInterface

webrtc/media/base/videosinkinterface.h

 /*
  *  Copyright (c) 2016 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.
  */
 
 #ifndef WEBRTC_MEDIA_BASE_VIDEOSINKINTERFACE_H_
 #define WEBRTC_MEDIA_BASE_VIDEOSINKINTERFACE_H_
 
 namespace rtc {
 
+struct VideoSinkCapabilities {
+  bool equals(const VideoSinkCapabilities& rh) const {
+    return can_apply_rotation == rh.can_apply_rotation;
+  }

[pthatcher1, 2016/02/03 15:38:36]

Why not use operator== ?

[perkj_webrtc, 2016/02/08 14:32:01]

because of style guide rules. But I think that is a stupid rule...

+
+  // Tells the source whether the sink can handle rotation. By default, the
+  // rotation is applied by the source and the generated frame is up right.
+  bool can_apply_rotation = false;
+};
+
 template <typename VideoFrameT>
 class VideoSinkInterface {
  public:
   virtual void OnFrame(const VideoFrameT& frame) = 0;
-
  protected:
-  ~VideoSinkInterface() {}
+  virtual ~VideoSinkInterface() {}
 };
 
 }  // namespace rtc
 
 #endif  // WEBRTC_MEDIA_BASE_VIDEOSINKINTERFACE_H_