Decoupling video_capture_internal_imlp from Obj-C code.

webrtc/modules/video_capture/BUILD.gn

 # Copyright (c) 2014 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")
 
@@ skipping 68 matching lines @@
         "-Wno-ignored-qualifiers",
         "-Wno-microsoft-extra-qualification",
         "-Wno-missing-braces",
         "-Wno-overloaded-virtual",
         "-Wno-reorder",
         "-Wno-writable-strings",
       ]
     }
   }
 
+  if (is_ios || is_mac) {
+    rtc_static_library("objc_video_capture_internal_impl") {

[kjellander_webrtc, 2017/08/14 07:00:39]

video_capture_internal_impl_objc

[mbonadei, 2017/08/15 14:01:06]

Done + added private visibility.

+      configs += [ ":video_capture_internal_impl_warnings_config" ]
+
+      deps = [
+        ":video_capture_module",
+        "../../rtc_base:rtc_base_approved",
+        "../../system_wrappers",
+      ]
+
+      sources = [
+        "objc/device_info.h",
+        "objc/device_info.mm",
+        "objc/device_info_objc.h",
+        "objc/device_info_objc.mm",
+        "objc/rtc_video_capture_objc.h",
+        "objc/rtc_video_capture_objc.mm",
+        "objc/video_capture.h",
+        "objc/video_capture.mm",
+      ]
+
+      all_dependent_configs = [ ":video_capture_internal_impl_config" ]
+
+      if (!build_with_chromium && is_clang) {
+        # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
+        suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
+      }
+    }
+  }
+
   rtc_static_library("video_capture_internal_impl") {
     configs += [ ":video_capture_internal_impl_warnings_config" ]
 
     deps = [
       ":video_capture_module",
       "../../rtc_base:rtc_base_approved",
       "../../system_wrappers",
     ]
 
     if (is_linux) {
@@ skipping 21 matching lines @@
         "windows/video_capture_mf.cc",
         "windows/video_capture_mf.h",
       ]
 
       libs = [ "Strmiids.lib" ]
 
       deps += [ "//third_party/winsdk_samples" ]
     }
     if (is_ios || is_mac) {
       sources = [
-        "objc/device_info.h",
-        "objc/device_info.mm",
-        "objc/device_info_objc.h",
-        "objc/device_info_objc.mm",
-        "objc/rtc_video_capture_objc.h",
-        "objc/rtc_video_capture_objc.mm",
-        "objc/video_capture.h",
-        "objc/video_capture.mm",
+        "objc/noop.cc",
       ]
+      deps += [ ":objc_video_capture_internal_impl" ]
     }
 
     all_dependent_configs = [ ":video_capture_internal_impl_config" ]
 
     if (!build_with_chromium && is_clang) {
       # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
       suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
     }
   }
 
@@ skipping 36 matching lines @@
       ]
       deps += [ "../../test:test_main" ]
 
       if (!build_with_chromium && is_clang) {
         # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
         suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
       }
     }
   }
 }