Delete IsMutable and IsExclusive methods.

webrtc/common_video/video_frame.cc

Index: webrtc/common_video/video_frame.cc
diff --git a/webrtc/common_video/video_frame.cc b/webrtc/common_video/video_frame.cc
index 4bd23f98dc06f796029f8fd4813355b91d11a813..978dde15b75eaa779a261d3c0636058e38556ef0 100644
--- a/webrtc/common_video/video_frame.cc
+++ b/webrtc/common_video/video_frame.cc
@@ -66,18 +66,7 @@ void VideoFrame::CreateEmptyFrame(int width,
   render_time_ms_ = 0;
   rotation_ = kVideoRotation_0;
 
-  // Check if it's safe to reuse allocation.
-  if (video_frame_buffer_ && video_frame_buffer_->IsMutable() &&
-      !video_frame_buffer_->native_handle() &&
-      width == video_frame_buffer_->width() &&
-      height == video_frame_buffer_->height() &&
-      stride_y == video_frame_buffer_->StrideY() &&
-      stride_u == video_frame_buffer_->StrideU() &&
-      stride_v == video_frame_buffer_->StrideV()) {
-    return;
-  }
-
-  // Need to allocate new buffer.
+  // Allocate a new buffer.

[pbos-webrtc, 2016/05/25 15:26:07]

Should this check if video_frame_buffer_->HasOneRef() instead of IsMutable? Is
this method called often or going away?

[nisse-webrtc, 2016/05/26 07:44:42]

It was used by the frame factory, that's why I changed it to make it use a
buffer pool instead. Besides that, the method is used only by test code.

And both HasOneRef and IsMutable are being purged from the public interface. (If
you want to keep one, IsMutable is the better name, imo).

   video_frame_buffer_ = new rtc::RefCountedObject<I420Buffer>(
       width, height, stride_y, stride_u, stride_v);
 }