RTCStats and RTCStatsReport added (webrtc/stats)

webrtc/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.
 
 # TODO(kjellander): Rebase this to webrtc/build/common.gypi changes after r6330.
 
@@ skipping 241 matching lines @@
     public_configs = [ ":common_inherited_config" ]
 
     deps = [
       ":webrtc_common",
       "audio",
       "base:rtc_base",
       "call",
       "common_audio",
       "common_video",
       "modules",
+      "stats:rtc_stats",

[hbos, 2016/08/13 08:31:39]

Should I list the unittest (rtc_stats_unittests) too, here or somewhere else? In
all.gyp wildcards are listed (e.g. stats.gyp:*) but that is not he trend here.
In any case, rtc_stats_unittests is globally known without explicitly being
listed.

[kjellander_webrtc, 2016/08/15 11:21:51]

If you just put "stats" here, GN will make the assumption that you have a
'stats' target inside the webrtc/stats/BUILD.gn (I even think it'll complain if
one is missing). 
I suggest you follow the pattern with a group entry as in
https://chromium.googlesource.com/external/webrtc/+/master/webrtc/media/BUILD.gn.

Since we don't want to include our test targets in Chromium builds, make sure to
just have it inside a rtc_include_tests condition, then it'll be processed (as
you've already noticed)

[hbos, 2016/08/15 13:27:11]

Done.

       "system_wrappers",
       "tools",
       "video",
       "voice_engine",
     ]
 
     if (build_with_chromium) {
       deps += [ "modules/video_capture" ]
     } else {
       # TODO(kjellander): Enable for Chromium as well when bugs.webrtc.org/4256
@@ skipping 494 matching lines @@
     if (is_ios || (is_mac && target_cpu != "x86")) {
       defines = [ "CARBON_DEPRECATED=YES" ]
     }
     if (is_clang) {
       # Suppress warnings from the Chromium Clang plugin.
       # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
       configs -= [ "//build/config/clang:find_bad_constructs" ]
     }
   }
 }