Moving webrtc/stats/ into webrtc/api/ to resolve a circular dependency

webrtc/api/rtcstatscollector.cc

 /*
  *  Copyright 2016 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.
  */
 

[nisse-webrtc, 2016/09/13 08:16:40]

Hmm, is this the move you're concerned about?

I've never quite understood our directory structure, hope kjellander can say if
this is right or not.

[hbos, 2016/09/13 08:50:38]

Yes, the stats -> api move (of not just that file but everything under
webrtc/stats/),
There's a doc[1] describing wanting to split up webrtc in smaller modules.
Putting stats in a separate folder and module made sense but then I hit this
dependency problem. Doing this move is going against splitting things up into
modules.

[1]
https://docs.google.com/document/d/1ZhugZLVk6ptbujZsnnpXUwqZS97L-gKM5VD9lGK_p...

-#include "webrtc/stats/rtcstatscollector.h"
+#include "webrtc/api/rtcstatscollector.h"
 
 #include <memory>
 #include <utility>
 #include <vector>
 
 #include "webrtc/api/peerconnection.h"
 #include "webrtc/base/checks.h"
 #include "webrtc/base/timing.h"
 
 namespace webrtc {
@@ skipping 150 matching lines @@
   // constant.
   std::unique_ptr<RTCPeerConnectionStats> stats(
     new RTCPeerConnectionStats("RTCPeerConnection", timestamp_us));
   stats->data_channels_opened = data_channels_opened;
   stats->data_channels_closed = static_cast<uint32_t>(data_channels.size()) -
                                 data_channels_opened;
   return stats;
 }
 
 }  // namespace webrtc