Delete cricket::VideoFrame methods GetYPlane and GetYPitch.

talk/app/webrtc/objc/RTCI420Frame.mm

 /*
  * libjingle
  * Copyright 2013 Google Inc.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are met:
  *
  *  1. Redistributions of source code must retain the above copyright notice,
  *     this list of conditions and the following disclaimer.
  *  2. Redistributions in binary form must reproduce the above copyright notice,
@@ skipping 38 matching lines @@
 - (NSUInteger)chromaWidth {
   return (self.width + 1) / 2;
 }
 
 - (NSUInteger)chromaHeight {
   return (self.height + 1) / 2;
 }
 
 - (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
 
 @implementation RTCI420Frame (Internal)
 
 - (instancetype)initWithVideoFrame:(cricket::VideoFrame*)videoFrame {
   if (self = [super init]) {
     // Keep a shallow copy of the video frame. The underlying frame buffer is
     // not copied.
     _videoFrame.reset(videoFrame->Copy());
   }
   return self;
 }
 
 @end