commit 25e022fd5c00914af5d7dde472260d0a6adb334c
author Taylor Brandstetter <deadbeef@webrtc.org> Thu Mar 08 09:53:47 2018 -0800
committer Commit Bot <commit-bot@chromium.org> Thu Mar 08 19:12:43 2018 +0000
tree 06308a6ad837d704aa7e83cce10490367594ff3a
parent 6328d7cbbc8a72fdc81a766c0bf4039e1e2e7887

Deliver cached stats reports asynchronously.

This has the following benefits:
* Stats reports are always delivered asynchronously. This means the API
  client doesn't need to worry about *possibly* getting a synchronous
  callback depending on when the last report was generated.
* Stats callbacks will always be invoked in the same order that the
  GetStats calls were made, even in cases where a callback recursively
  calls GetStats again.

Bug: webrtc:8973
Change-Id: I94ca4b5dc5c21a8f2df42adfcddf357f40a32025
Reviewed-on: https://webrtc-review.googlesource.com/60473
Commit-Queue: Taylor Brandstetter <deadbeef@webrtc.org>
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#22348}

pc/rtcstatscollector.h

diff --git a/pc/rtcstatscollector.h b/pc/rtcstatscollector.h
index 2ff4c49..be5980b 100644
--- a/pc/rtcstatscollector.h
+++ b/pc/rtcstatscollector.h
@@ -94,7 +94,9 @@
   };
 
   void AddPartialResults_s(rtc::scoped_refptr<RTCStatsReport> partial_report);
-  void DeliverCachedReport();
+  void DeliverCachedReport(
+      rtc::scoped_refptr<const RTCStatsReport> cached_report,
+      std::vector<rtc::scoped_refptr<RTCStatsCollectorCallback>> callbacks);
 
   // Produces |RTCCertificateStats|.
   void ProduceCertificateStats_n(