Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1164)

Unified Diff: webrtc/api/rtcstatscollector_unittest.cc

Issue 2521663002: RTCStatsIntegrationTest added. (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/api/rtcstatscollector_unittest.cc
diff --git a/webrtc/api/rtcstatscollector_unittest.cc b/webrtc/api/rtcstatscollector_unittest.cc
index 64e7e0a97b18dd95fec64a20323b5ca860c65930..3187da84151a5839847156a294bea13e4dea9985 100644
--- a/webrtc/api/rtcstatscollector_unittest.cc
+++ b/webrtc/api/rtcstatscollector_unittest.cc
@@ -23,6 +23,7 @@
#include "webrtc/api/test/mock_datachannel.h"
#include "webrtc/api/test/mock_peerconnection.h"
#include "webrtc/api/test/mock_webrtcsession.h"
+#include "webrtc/api/test/rtcstatsobtainer.h"
#include "webrtc/base/checks.h"
#include "webrtc/base/fakeclock.h"
#include "webrtc/base/fakesslidentity.h"
@@ -467,37 +468,6 @@ class FakeRTCStatsCollector : public RTCStatsCollector,
int produced_on_network_thread_ = 0;
};
-class StatsCallback : public RTCStatsCollectorCallback {
- public:
- static rtc::scoped_refptr<StatsCallback> Create(
- rtc::scoped_refptr<const RTCStatsReport>* report_ptr = nullptr) {
- return rtc::scoped_refptr<StatsCallback>(
- new rtc::RefCountedObject<StatsCallback>(report_ptr));
- }
-
- void OnStatsDelivered(
- const rtc::scoped_refptr<const RTCStatsReport>& report) override {
- EXPECT_TRUE(thread_checker_.CalledOnValidThread());
- report_ = report;
- if (report_ptr_)
- *report_ptr_ = report_;
- }
-
- rtc::scoped_refptr<const RTCStatsReport> report() const {
- EXPECT_TRUE(thread_checker_.CalledOnValidThread());
- return report_;
- }
-
- protected:
- explicit StatsCallback(rtc::scoped_refptr<const RTCStatsReport>* report_ptr)
- : report_ptr_(report_ptr) {}
-
- private:
- rtc::ThreadChecker thread_checker_;
- rtc::scoped_refptr<const RTCStatsReport> report_;
- rtc::scoped_refptr<const RTCStatsReport>* report_ptr_;
-};
-
class RTCStatsCollectorTest : public testing::Test {
public:
RTCStatsCollectorTest()
@@ -507,7 +477,7 @@ class RTCStatsCollectorTest : public testing::Test {
}
rtc::scoped_refptr<const RTCStatsReport> GetStatsReport() {
- rtc::scoped_refptr<StatsCallback> callback = StatsCallback::Create();
+ rtc::scoped_refptr<RTCStatsObtainer> callback = RTCStatsObtainer::Create();
collector_->GetStatsReport(callback);
EXPECT_TRUE_WAIT(callback->report(), kGetStatsReportTimeoutMs);
int64_t after = rtc::TimeUTCMicros();
@@ -718,7 +688,7 @@ class RTCStatsCollectorTest : public testing::Test {
TEST_F(RTCStatsCollectorTest, SingleCallback) {
rtc::scoped_refptr<const RTCStatsReport> result;
- collector_->GetStatsReport(StatsCallback::Create(&result));
+ collector_->GetStatsReport(RTCStatsObtainer::Create(&result));
EXPECT_TRUE_WAIT(result, kGetStatsReportTimeoutMs);
}
@@ -726,9 +696,9 @@ TEST_F(RTCStatsCollectorTest, MultipleCallbacks) {
rtc::scoped_refptr<const RTCStatsReport> a;
rtc::scoped_refptr<const RTCStatsReport> b;
rtc::scoped_refptr<const RTCStatsReport> c;
- collector_->GetStatsReport(StatsCallback::Create(&a));
- collector_->GetStatsReport(StatsCallback::Create(&b));
- collector_->GetStatsReport(StatsCallback::Create(&c));
+ collector_->GetStatsReport(RTCStatsObtainer::Create(&a));
+ collector_->GetStatsReport(RTCStatsObtainer::Create(&b));
+ collector_->GetStatsReport(RTCStatsObtainer::Create(&c));
EXPECT_TRUE_WAIT(a, kGetStatsReportTimeoutMs);
EXPECT_TRUE_WAIT(b, kGetStatsReportTimeoutMs);
EXPECT_TRUE_WAIT(c, kGetStatsReportTimeoutMs);
@@ -756,11 +726,11 @@ TEST_F(RTCStatsCollectorTest, MultipleCallbacksWithInvalidatedCacheInBetween) {
rtc::scoped_refptr<const RTCStatsReport> a;
rtc::scoped_refptr<const RTCStatsReport> b;
rtc::scoped_refptr<const RTCStatsReport> c;
- collector_->GetStatsReport(StatsCallback::Create(&a));
- collector_->GetStatsReport(StatsCallback::Create(&b));
+ collector_->GetStatsReport(RTCStatsObtainer::Create(&a));
+ collector_->GetStatsReport(RTCStatsObtainer::Create(&b));
// Cache is invalidated after 50 ms.
test_->fake_clock().AdvanceTime(rtc::TimeDelta::FromMilliseconds(51));
- collector_->GetStatsReport(StatsCallback::Create(&c));
+ collector_->GetStatsReport(RTCStatsObtainer::Create(&c));
EXPECT_TRUE_WAIT(a, kGetStatsReportTimeoutMs);
EXPECT_TRUE_WAIT(b, kGetStatsReportTimeoutMs);
EXPECT_TRUE_WAIT(c, kGetStatsReportTimeoutMs);

Powered by Google App Engine
This is Rietveld 408576698