commit fb372f0074a0cb21fa36976316c0b5fe3b4ae7bc
author magjed <magjed@webrtc.org> Wed Aug 10 07:58:29 2016 -0700
committer Commit bot <commit-bot@chromium.org> Wed Aug 10 14:58:35 2016 +0000
tree 3c4fe7997b4e70c78e4ed1fa0ebfbcb2d6e9d64b
parent 4556b450765116072f9700e0619c86b0f9b06588

iOS render: Handle frame rotation in OpenGL

This CL handles frame rotation by updating the OpenGL vertex data in
RTCOpenGLVideoRenderer, instead of calling the expensive
libyuv::I420Rotate that will rotate the actual memory. Also, we can
handle rotated native frames instead of falling back to
NativeToI420Buffer.

Review-Url: https://codereview.webrtc.org/2176623002
Cr-Commit-Position: refs/heads/master@{#13715}

webrtc/sdk/objc/Framework/Classes/RTCNativeNV12Shader.mm

diff --git a/webrtc/sdk/objc/Framework/Classes/RTCNativeNV12Shader.mm b/webrtc/sdk/objc/Framework/Classes/RTCNativeNV12Shader.mm
index cc97943..75a575b 100644
--- a/webrtc/sdk/objc/Framework/Classes/RTCNativeNV12Shader.mm
+++ b/webrtc/sdk/objc/Framework/Classes/RTCNativeNV12Shader.mm
@@ -20,6 +20,8 @@
 #import "WebRTC/RTCVideoFrame.h"
 
 #include "webrtc/base/checks.h"
+#include "webrtc/base/optional.h"
+#include "webrtc/common_video/rotation.h"
 
 static const char kNV12FragmentShaderSource[] =
   SHADER_VERSION
@@ -46,6 +48,9 @@
   GLint _ySampler;
   GLint _uvSampler;
   CVOpenGLESTextureCacheRef _textureCache;
+  // Store current rotation and only upload new vertex data when rotation
+  // changes.
+  rtc::Optional<webrtc::VideoRotation> _currentRotation;
 }
 
 - (instancetype)initWithContext:(GlContextType *)context {
@@ -149,6 +154,11 @@
   glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_CLAMP_TO_EDGE);
 
   glBindBuffer(GL_ARRAY_BUFFER, _vertexBuffer);
+  if (!_currentRotation || frame.rotation != *_currentRotation) {
+    _currentRotation = rtc::Optional<webrtc::VideoRotation>(
+        static_cast<webrtc::VideoRotation>(frame.rotation));
+    RTCSetVertexData(*_currentRotation);
+  }
   glDrawArrays(GL_TRIANGLE_FAN, 0, 4);
 
   CFRelease(chromaTexture);