From 754c952dfa519a3bc956b050a70b2291335aeb74 Mon Sep 17 00:00:00 2001
From: Artem Titov <titovartem@webrtc.org>
Date: Mon, 1 Jul 2019 13:38:46 +0200
Subject: [PATCH] Don't do ToI420() for each frame while checking is it dummy

Bug: webrtc:10138
Change-Id: I7dba58324afd65ce046a0a255277de7134ebd0b7
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/144242
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#28429}
---
 test/pc/e2e/analyzer/video/simulcast_dummy_buffer_helper.cc | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/test/pc/e2e/analyzer/video/simulcast_dummy_buffer_helper.cc b/test/pc/e2e/analyzer/video/simulcast_dummy_buffer_helper.cc
index d0548cbbbf..bfe5e57be7 100644
--- a/test/pc/e2e/analyzer/video/simulcast_dummy_buffer_helper.cc
+++ b/test/pc/e2e/analyzer/video/simulcast_dummy_buffer_helper.cc
@@ -32,11 +32,11 @@ rtc::scoped_refptr<webrtc::VideoFrameBuffer> CreateDummyFrameBuffer() {
 
 bool IsDummyFrameBuffer(
     rtc::scoped_refptr<webrtc::VideoFrameBuffer> video_frame_buffer) {
-  rtc::scoped_refptr<webrtc::I420BufferInterface> buffer =
-      video_frame_buffer->ToI420();
-  if (buffer->width() != 2 || buffer->height() != 2) {
+  if (video_frame_buffer->width() != 2 || video_frame_buffer->height() != 2) {
     return false;
   }
+  rtc::scoped_refptr<webrtc::I420BufferInterface> buffer =
+      video_frame_buffer->ToI420();
   if (memcmp(buffer->DataY(), kIrrelatedSimulcastStreamFrameData, 2) != 0) {
     return false;
   }