Reland of Delete cricket::VideoFrame methods GetYPlane and GetYPitch.

webrtc/media/base/videoframe.h

 /*
  *  Copyright (c) 2004 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.
  */
 
@@ skipping 17 matching lines @@
   // Note this is the width and height without rotation applied.
   virtual int width() const = 0;
   virtual int height() const = 0;
 
   // Deprecated methods, for backwards compatibility.
   // TODO(nisse): Delete when usage in Chrome and other applications
   // have been replaced by width() and height().
   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.
   virtual void* GetNativeHandle() const = 0;
 
   // 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;
 
   // System monotonic clock, same timebase as rtc::TimeMicros().
   virtual int64_t timestamp_us() const = 0;
   virtual void set_timestamp_us(int64_t time_us) = 0;
 
   // Deprecated methods, for backwards compatibility.
   // TODO(nisse): Delete when usage in Chrome and other applications
   // have been replaced.
   virtual int64_t GetTimeStamp() const {
     return rtc::kNumNanosecsPerMicrosec * timestamp_us();
@@ skipping 81 matching lines @@
   // Creates an empty frame.
   virtual VideoFrame* CreateEmptyFrame(int w,
                                        int h,
                                        int64_t timestamp_us) const = 0;
   virtual void set_rotation(webrtc::VideoRotation rotation) = 0;
 };
 
 }  // namespace cricket
 
 #endif  // WEBRTC_MEDIA_BASE_VIDEOFRAME_H_