commit 4f7b6406c469415590c682af50963587ce2f5071
author Kári Tristan Helgason <kthelgason@webrtc.org> Mon Apr 16 11:39:53 2018 +0200
committer Commit Bot <commit-bot@chromium.org> Thu Apr 19 11:07:39 2018 +0000
tree 9d3270921481101218e3107e6e329932c1b05537
parent aedd090b72b2fa3d8a8884c5ad647741de27be07

Add checks that we don't redraw the previous frame.

Bug: webrtc:9149
Change-Id: Ia1f61fd43ea9be6c341a111595e8a290a809c72f
Reviewed-on: https://webrtc-review.googlesource.com/69810
Reviewed-by: Peter Hanspers <peterhanspers@webrtc.org>
Commit-Queue: Kári Helgason <kthelgason@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#22937}

sdk/objc/Framework/Classes/Metal/RTCMTLVideoView.m

diff --git a/sdk/objc/Framework/Classes/Metal/RTCMTLVideoView.m b/sdk/objc/Framework/Classes/Metal/RTCMTLVideoView.m
index 0fd8a1a..24dc29f 100644
--- a/sdk/objc/Framework/Classes/Metal/RTCMTLVideoView.m
+++ b/sdk/objc/Framework/Classes/Metal/RTCMTLVideoView.m
@@ -33,7 +33,9 @@
 @property(atomic, strong) RTCVideoFrame *videoFrame;
 @end
 
-@implementation RTCMTLVideoView
+@implementation RTCMTLVideoView {
+  int64_t _lastFrameTimeNs;
+}
 
 @synthesize rendererI420 = _rendererI420;
 @synthesize rendererNV12 = _rendererNV12;
@@ -106,7 +108,8 @@
 - (void)drawInMTKView:(nonnull MTKView *)view {
   NSAssert(view == self.metalView, @"Receiving draw callbacks from foreign instance.");
   RTCVideoFrame *videoFrame = self.videoFrame;
-  if (!videoFrame) {
+  // Skip rendering if we've already rendered this frame.
+  if (!videoFrame || videoFrame.timeStampNs == _lastFrameTimeNs) {
     return;
   }
 
@@ -116,6 +119,7 @@
       if (![self.rendererNV12 addRenderingDestination:self.metalView]) {
         self.rendererNV12 = nil;
         RTCLogError(@"Failed to create NV12 renderer");
+        return;
       }
     }
     [self.rendererNV12 drawFrame:videoFrame];
@@ -125,10 +129,12 @@
       if (![self.rendererI420 addRenderingDestination:self.metalView]) {
         self.rendererI420 = nil;
         RTCLogError(@"Failed to create I420 renderer");
+        return;
       }
     }
     [self.rendererI420 drawFrame:videoFrame];
   }
+  _lastFrameTimeNs = videoFrame.timeStampNs;
 }
 
 - (void)mtkView:(MTKView *)view drawableSizeWillChange:(CGSize)size {