Move array_view.h to webrtc/api/

webrtc/api/BUILD.gn

Index: webrtc/api/BUILD.gn
diff --git a/webrtc/api/BUILD.gn b/webrtc/api/BUILD.gn
index 3d19f03649e37822dc5ec2ae9039b8c84da804f7..1980c29ca9d4c78374982b9d60f70abef399b47c 100644
--- a/webrtc/api/BUILD.gn
+++ b/webrtc/api/BUILD.gn
@@ -197,6 +197,15 @@ rtc_source_set("video_frame_api") {
   }
 }
 
+rtc_source_set("array_view") {
+  sources = [
+    "array_view.h",
+  ]
+  deps = [
+    "../rtc_base:rtc_base_approved",
+  ]
+}
+
 rtc_source_set("libjingle_peerconnection_test_api") {
   testonly = true
   sources = [
@@ -255,6 +264,7 @@ if (rtc_include_tests) {
       visibility = [ "..:rtc_unittests" ]
     }
     sources = [
+      "array_view_unittest.cc",
       "ortc/mediadescription_unittest.cc",
       "ortc/sessiondescription_unittest.cc",
       "rtcerror_unittest.cc",
@@ -264,10 +274,18 @@ if (rtc_include_tests) {
       # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
       suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
     }
+    defines = [ "GTEST_RELATIVE_PATH" ]

[mbonadei, 2017/09/01 08:08:34]

I think we can avoid to define GTEST_RELATIVE_PATH because it is defined in the
public_config attribute of rtc_base:rtc_base_tests_utils and it gets inherited.

[kwiberg-webrtc, 2017/09/01 08:58:02]

Done.

+
+    public_deps = [

[mbonadei, 2017/09/01 08:08:34]

Also //testing/gmock is listed in the public_deps attribute of
rtc_base:rtc_base_tests_utils so we can remove this (unless we want to be
explicit but in that case I think we can move it to deps and avoid to re-expose
it to targets that will depend on :rtc_api_unittests).

[kwiberg-webrtc, 2017/09/01 08:58:02]

Done.

+      "//testing/gmock",
+    ]
 
     deps = [
+      ":array_view",
       ":libjingle_peerconnection_api",
       ":ortc_api",
+      "../rtc_base:rtc_base_approved",
+      "../rtc_base:rtc_base_tests_utils",
       "../test:test_support",
     ]
   }