Delete cricket::VideoFrame methods GetYPlane and GetYPitch.

webrtc/media/base/videoframe.h

Index: webrtc/media/base/videoframe.h
diff --git a/webrtc/media/base/videoframe.h b/webrtc/media/base/videoframe.h
index 9e0fbfd6d42ceff6f036935392b06dad54bee4d3..4026c26b3b136ab26229ab884c249e4d38a15aba 100644
--- a/webrtc/media/base/videoframe.h
+++ b/webrtc/media/base/videoframe.h
@@ -35,18 +35,6 @@ class VideoFrame {
   virtual size_t GetWidth() const final { return width(); }
   virtual size_t GetHeight() const final { return height(); }
 
-  // These can return NULL if the object is not backed by a buffer.
-  virtual const uint8_t* GetYPlane() const = 0;
-  virtual const uint8_t* GetUPlane() const = 0;
-  virtual const uint8_t* GetVPlane() const = 0;
-  virtual uint8_t* GetYPlane() = 0;
-  virtual uint8_t* GetUPlane() = 0;
-  virtual uint8_t* GetVPlane() = 0;
-
-  virtual int32_t GetYPitch() const = 0;
-  virtual int32_t GetUPitch() const = 0;
-  virtual int32_t GetVPitch() const = 0;
-
   // Returns the handle of the underlying video frame. This is used when the
   // frame is backed by a texture. The object should be destroyed when it is no
   // longer in use, so the underlying resource can be freed.
@@ -54,8 +42,8 @@ class VideoFrame {
 
   // Returns the underlying video frame buffer. This function is ok to call
   // multiple times, but the returned object will refer to the same memory.
-  virtual rtc::scoped_refptr<webrtc::VideoFrameBuffer> video_frame_buffer()
-      const = 0;
+  virtual const rtc::scoped_refptr<webrtc::VideoFrameBuffer>&
+  video_frame_buffer() const = 0;

[perkj_webrtc, 2016/04/25 14:24:09]

git cl format?

[nisse-webrtc, 2016/04/25 14:27:13]

git cl format likes it like that. I agree it looks strange.

 
   // System monotonic clock, same timebase as rtc::TimeMicros().
   virtual int64_t timestamp_us() const = 0;