Eliminate calls to cricket::VideoFrame's GetNativeHandle, GetYPlane.

content/renderer/media/webrtc/media_stream_remote_video_source.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/media/webrtc/media_stream_remote_video_source.h"
 
 #include <stdint.h>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 75 matching lines @@
   TRACE_EVENT1("webrtc", "RemoteVideoSourceDelegate::RenderFrame",
                "Ideal Render Instant", render_time.ToInternalValue());
 
   CHECK_NE(media::kNoTimestamp(), incoming_timestamp);
   if (start_timestamp_ == media::kNoTimestamp())
     start_timestamp_ = incoming_timestamp;
   const base::TimeDelta elapsed_timestamp =
       incoming_timestamp - start_timestamp_;
 
   scoped_refptr<media::VideoFrame> video_frame;
-  if (incoming_frame.GetNativeHandle() != NULL) {
+  if (incoming_frame.video_frame_buffer()->native_handle() != NULL) {
     video_frame =
-        static_cast<media::VideoFrame*>(incoming_frame.GetNativeHandle());
+        static_cast<media::VideoFrame*>(
+            incoming_frame.video_frame_buffer()->native_handle());
     video_frame->set_timestamp(elapsed_timestamp);
   } else {
     const cricket::VideoFrame* frame =
         incoming_frame.GetCopyWithRotationApplied();
 
     gfx::Size size(frame->width(), frame->height());
 
     // Make a shallow copy. Both |frame| and |video_frame| will share a single
     // reference counted frame buffer. Const cast and hope no one will overwrite
     // the data.
     // TODO(magjed): Update media::VideoFrame to support const data so we don't
     // need to const cast here.
     video_frame = media::VideoFrame::WrapExternalYuvData(
         media::PIXEL_FORMAT_YV12, size, gfx::Rect(size), size,
-        frame->GetYPitch(), frame->GetUPitch(), frame->GetVPitch(),
-        const_cast<uint8_t*>(frame->GetYPlane()),
-        const_cast<uint8_t*>(frame->GetUPlane()),
-        const_cast<uint8_t*>(frame->GetVPlane()), elapsed_timestamp);
+        frame->video_frame_buffer()->StrideY(),
+        frame->video_frame_buffer()->StrideU(),
+        frame->video_frame_buffer()->StrideV(),
+        const_cast<uint8_t*>(frame->video_frame_buffer()->DataY()),
+        const_cast<uint8_t*>(frame->video_frame_buffer()->DataU()),
+        const_cast<uint8_t*>(frame->video_frame_buffer()->DataV()),
+        elapsed_timestamp);
     if (!video_frame)
       return;
     video_frame->AddDestructionObserver(
         base::Bind(&base::DeletePointer<cricket::VideoFrame>, frame->Copy()));
   }
 
   video_frame->metadata()->SetTimeTicks(
       media::VideoFrameMetadata::REFERENCE_TIME, render_time);
 
   io_task_runner_->PostTask(
@@ skipping 85 matching lines @@
     case webrtc::MediaStreamTrackInterface::kEnded:
       SetReadyState(blink::WebMediaStreamSource::ReadyStateEnded);
       break;
     default:
       NOTREACHED();
       break;
   }
 }
 
 }  // namespace content