Delete cricket::VideoFrame methods GetYPlane and GetYPitch.

talk/app/webrtc/objc/RTCI420Frame.mm

Index: talk/app/webrtc/objc/RTCI420Frame.mm
diff --git a/talk/app/webrtc/objc/RTCI420Frame.mm b/talk/app/webrtc/objc/RTCI420Frame.mm
index aacdfe33b22fb387a659425c7208d8a2b78c3191..5167e0741a35fd43e9faec4ea113833d9cb4aa01 100644
--- a/talk/app/webrtc/objc/RTCI420Frame.mm
+++ b/talk/app/webrtc/objc/RTCI420Frame.mm
@@ -56,29 +56,29 @@
 
 - (const uint8_t*)yPlane {
   const cricket::VideoFrame* const_frame = _videoFrame.get();
-  return const_frame->GetYPlane();
+  return const_frame->video_frame_buffer()->DataY();

[magjed_webrtc, 2016/04/19 11:50:52]

This changes the behavior of this function, it used to return null if
video_frame_buffer() was null, now it will crash. Are you sure it will work?
Also, you don't need the const cast to above anymore, use
videoFrame->video_frame_buffer()->DataY() directly instead.

[nisse-webrtc, 2016/04/20 07:21:53]

Don't really know, but webrtc/api/objc version, RTCVideoFrame, seems to tolerate
a NULL video frame buffer, so I'll do the same here.

 }
 
 - (const uint8_t*)uPlane {
   const cricket::VideoFrame* const_frame = _videoFrame.get();
-  return const_frame->GetUPlane();
+  return const_frame->video_frame_buffer()->DataU();
 }
 
 - (const uint8_t*)vPlane {
   const cricket::VideoFrame* const_frame = _videoFrame.get();
-  return const_frame->GetVPlane();
+  return const_frame->video_frame_buffer()->DataV();
 }
 
 - (NSInteger)yPitch {
-  return _videoFrame->GetYPitch();
+  return _videoFrame->video_frame_buffer()->StrideY();
 }
 
 - (NSInteger)uPitch {
-  return _videoFrame->GetUPitch();
+  return _videoFrame->video_frame_buffer()->StrideU();
 }
 
 - (NSInteger)vPitch {
-  return _videoFrame->GetVPitch();
+  return _videoFrame->video_frame_buffer()->StrideV();
 }
 
 @end