Eliminate use of GetCopyWithRotationApplied.

content/renderer/media/webrtc/media_stream_remote_video_source.cc

Index: content/renderer/media/webrtc/media_stream_remote_video_source.cc
diff --git a/content/renderer/media/webrtc/media_stream_remote_video_source.cc b/content/renderer/media/webrtc/media_stream_remote_video_source.cc
index 30c25baa3dbf659b70c4316d13b9aecdd8333b1c..8f1e0751df92974e28f4f87f23a793ccef1f5ea8 100644
--- a/content/renderer/media/webrtc/media_stream_remote_video_source.cc
+++ b/content/renderer/media/webrtc/media_stream_remote_video_source.cc
@@ -124,10 +124,9 @@ void MediaStreamRemoteVideoSource::RemoteVideoSourceDelegate::OnFrame(
           WebRTCToMediaVideoRotation(incoming_frame.rotation()));
     }
   } else {
-    // Note that the GetCopyWithRotationApplied returns a pointer to a
-    // frame owned by incoming_frame.
-    buffer =
-        incoming_frame.GetCopyWithRotationApplied()->video_frame_buffer();
+    buffer = webrtc::I420Buffer::Rotate(incoming_frame.video_frame_buffer(),

[perkj_chrome, 2016/09/01 06:44:28]

Can you check if video_frame->metadata()->SetRotation() can be used here
instead?  That will probably save a bit of processing but it requires all use
case to support it. Ie, rendering, recording, and sending this frame on a pc
again. 

Might want to file a separate bug on this if it is intended to work.

[nisse-chromium (ooo August 14), 2016/09/01 07:09:09]

There's only one call to GetRotation, in WebMediaPlayerMS. There, it is passed
on to the VideoLayer and VideoLayerImpl classes. Unclear how it is used, I tried
hardcoding the rotation to media::VIDEO_ROTATION_90 here, but that doesn't seem
to have any effect (tested with
https://webrtc.github.io/samples/src/content/peerconnection/pc1/).

+                                        incoming_frame.rotation());
+
     gfx::Size size(buffer->width(), buffer->height());
 
     // Make a shallow copy. Both |frame| and |video_frame| will share a single