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

webrtc/api/BUILD.gn

 # Copyright (c) 2015 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")
 if (is_android) {
@@ skipping 323 matching lines @@
 rtc_source_set("audio_mixer_api") {
   sources = [
     "audio/audio_mixer.h",
   ]
 
   deps = [
     "../base:rtc_base_approved",
   ]
 }
 
+rtc_source_set("video_frame_api") {
+  sources = [
+    "video/rotation.h",

[pthatcher1, 2016/11/30 00:48:28]

We have video/video_x.h for everything else.  Wouldn't video/video_rotation.h be
more consistent?

[nisse-webrtc, 2016/11/30 11:26:03]

I'm renaming it video/video_rotation.h, for consistency. It defines the (enum)
type VideoRotation.

+    "video/video_frame.cc",
+    "video/video_frame.h",
+    "video/video_frame_buffer.h",
+  ]
+
+  deps = [
+    "../base:rtc_base_approved",
+  ]
+}
+
 if (rtc_include_tests) {
   config("peerconnection_unittests_config") {
     # The warnings below are enabled by default. Since GN orders compiler flags
     # for a target before flags from configs, the only way to disable such
     # warnings is by having them in a separate config, loaded from the target.
     # TODO(kjellander): Make the code compile without disabling these flags.
     # See https://bugs.webrtc.org/3307.
     if (is_clang && is_win) {
       cflags = [
         # See https://bugs.chromium.org/p/webrtc/issues/detail?id=6267
@@ skipping 132 matching lines @@
       deps = [
         ":libjingle_peerconnection_java",
         "../base:base_java",
         "//base:base_java",
       ]
 
       shared_libraries = [ ":libjingle_peerconnection_so" ]
     }
   }
 }