Add unit tests for RTCMTLVideoView.

webrtc/sdk/BUILD.gn

 # Copyright 2016 The WebRTC project authors. All Rights Reserved.
 #
 # Use of this source code is governed by a BSD-style license
 # that can be found in the LICENSE file in the root of the source
 # tree. An additional intellectual property rights grant can be found
 # in the file PATENTS.  All contributing project authors may
 # be found in the AUTHORS file in the root of the source tree.
 
 import("../webrtc.gni")
 if (is_ios) {
@@ skipping 76 matching lines @@
     if (!build_with_chromium) {
       sources += [
         "objc/Framework/Classes/RTCFileLogger.mm",
         "objc/Framework/Headers/WebRTC/RTCFileLogger.h",
       ]
     }
   }
 
   rtc_static_library("rtc_sdk_peerconnection_objc") {
     sources = [
+      "objc/Framework/Classes/Metal/RTCMTLNV12Renderer.h",
       "objc/Framework/Classes/RTCAVFoundationVideoCapturerInternal.h",
       "objc/Framework/Classes/RTCAVFoundationVideoCapturerInternal.mm",
       "objc/Framework/Classes/RTCAVFoundationVideoSource+Private.h",
       "objc/Framework/Classes/RTCAVFoundationVideoSource.mm",
       "objc/Framework/Classes/RTCAudioSource+Private.h",
       "objc/Framework/Classes/RTCAudioSource.mm",
       "objc/Framework/Classes/RTCAudioTrack+Private.h",
       "objc/Framework/Classes/RTCAudioTrack.mm",
       "objc/Framework/Classes/RTCConfiguration+Private.h",
       "objc/Framework/Classes/RTCConfiguration.mm",
@@ skipping 139 matching lines @@
   }
 
   if (rtc_include_tests) {
     rtc_source_set("rtc_sdk_peerconnection_objc_unittests") {
       testonly = true
       sources = [
         "objc/Framework/UnitTests/RTCConfigurationTest.mm",
         "objc/Framework/UnitTests/RTCDataChannelConfigurationTest.mm",
         "objc/Framework/UnitTests/RTCIceCandidateTest.mm",
         "objc/Framework/UnitTests/RTCIceServerTest.mm",
+        "objc/Framework/UnitTests/RTCMTLVideoViewTests.mm",
         "objc/Framework/UnitTests/RTCMediaConstraintsTest.mm",
         "objc/Framework/UnitTests/RTCSessionDescriptionTest.mm",
         "objc/Framework/UnitTests/avformatmappertests.mm",
       ]
+      if (current_cpu != "arm64") {

[kthelgason, 2017/03/02 08:23:05]

Isn't this conditional the wrong way around?

+        sources += [ "objc/Framework/Classes/Metal/RTCMTLVideoView.m" ]

[magjed_webrtc, 2017/03/02 13:28:18]

Is it ok to always do:
sources += [ "objc/Framework/Classes/Metal/RTCMTLVideoView.m" ]
unconditionally or will that lead to an error on arm64 where it's added twice to
sources?

[daniela-webrtc, 2017/03/03 08:47:01]

Yes, it's added twice for arm64 in that case and compilation fails (duplicate
symbols)

+      }
       deps = [
         ":rtc_sdk_peerconnection_objc",
         "//third_party/ocmock",
         "//webrtc/system_wrappers:system_wrappers_default",
       ]
 
       # TODO(tkchin): Cleanup this warning.
       cflags = [ "-Wno-objc-property-no-attribute" ]
 
       # |-ObjC| flag needed to make sure category method implementations
@@ skipping 142 matching lines @@
       deps += [ "$rtc_libyuv_dir" ]
       public_deps = [
         "$rtc_libyuv_dir",
       ]
     } else {
       # Need to add a directory normally exported by libyuv.
       include_dirs = [ "$rtc_libyuv_dir/include" ]
     }
   }
 }