Move webrtc/video_frame to common_video/include.

webrtc/api/video_codecs/BUILD.gn

Index: webrtc/api/video_codecs/BUILD.gn
diff --git a/webrtc/api/video_codecs/BUILD.gn b/webrtc/api/video_codecs/BUILD.gn
index 5562eec659444652103d0cce3b5838a9714ae4b8..e6baed037e6112993abc8c353433039d2bcb26f8 100644
--- a/webrtc/api/video_codecs/BUILD.gn
+++ b/webrtc/api/video_codecs/BUILD.gn
@@ -19,11 +19,11 @@ rtc_source_set("video_codecs_api") {
   ]
 
   deps = [
-    # TODO(ilnik): Add dependency on webrtc/video_frame.h when it will have it's
-    # own build target.
-
     "..:video_frame_api",
     "../..:webrtc_common",
     "../../base:rtc_base_approved",
+
+    # Needed for EncodedImage.
+    "../../common_video:common_video",

[nisse-webrtc, 2017/05/11 07:13:34]

Is this dependency appropriate? I'm not sure what's really in video_codecs_api.

[kwiberg-webrtc, 2017/05/12 09:06:24]

I think the answer is "whatever is declared in video_{en,de}coder.h". (By the
way, it's suspicious that there are no .cc files listed in sources. Do they
really not declare anything that they don't define?)

However, I *can* tell you that you can drop the ":common_video" suffix, since
it's implied if you omit it.

[nisse-webrtc, 2017/05/12 10:42:19]

They declare interfaces only, it seems. Hence no source files. And the
interfaces depend on EncodedImage, so I guess this dependency is appropriate.
Deleted.

   ]
 }