Move VideoFrame and related declarations to webrtc/api/video.

webrtc/common_video/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("../build/webrtc.gni")
 
@@ skipping 42 matching lines @@
   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" ]
   }
 
   deps = [
     "..:webrtc_common",
     "../base:rtc_task_queue",
     "../system_wrappers",
   ]
+  public_deps = [
+    "../api:video_frame_api",
+  ]
 
   if (rtc_build_libyuv) {
     deps += [ "$rtc_libyuv_dir" ]

[nisse-webrtc, 2016/12/16 08:42:53]

Adding the same value to both deps and public_deps looks redundant?

[kjellander_webrtc, 2016/12/16 08:46:48]

Yeah you're right. But it's like that all over the codebase so better clean that
up in a separate CL.

-    public_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" ]
   }
 
   if (is_ios || is_mac) {
     sources += [
       "corevideo_frame_buffer.cc",
       "include/corevideo_frame_buffer.h",
     ]
@@ skipping 50 matching lines @@
     if (is_android) {
       deps += [ "//testing/android/native_test:native_test_support" ]
       shard_timeout = 900
     }
 
     if (is_ios) {
       deps += [ ":common_video_unittests_bundle_data" ]
     }
   }
 }