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

Unified Diff: chrome/browser/net/nqe/ui_network_quality_estimator_service_browsertest.cc

Issue 2742293004: Expose getters methods for NQE on UI thread (Closed)
Patch Set: ryansturm comments Created 3 years, 9 months 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
« no previous file with comments | « chrome/browser/net/nqe/ui_network_quality_estimator_service.cc ('k') | net/nqe/network_quality_estimator.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/net/nqe/ui_network_quality_estimator_service_browsertest.cc
diff --git a/chrome/browser/net/nqe/ui_network_quality_estimator_service_browsertest.cc b/chrome/browser/net/nqe/ui_network_quality_estimator_service_browsertest.cc
index e4119d5ddfb43bbb79e2e5c5130acfb467b6caea..72d41e6034b411f1a5d0214e7a643926c577ed3a 100644
--- a/chrome/browser/net/nqe/ui_network_quality_estimator_service_browsertest.cc
+++ b/chrome/browser/net/nqe/ui_network_quality_estimator_service_browsertest.cc
@@ -257,11 +257,19 @@ IN_PROC_BROWSER_TEST_F(UINetworkQualityEstimatorServiceBrowserTest,
nqe_test_util::OverrideRTTsAndWait(rtt_1);
EXPECT_EQ(rtt_1, nqe_observer.http_rtt());
+ EXPECT_EQ(rtt_1, nqe_service->GetHttpRTT());
+ EXPECT_EQ(rtt_1, nqe_service->GetTransportRTT());
+ EXPECT_FALSE(nqe_service->GetDownstreamThroughputKbps().has_value());
+
base::TimeDelta rtt_2 = base::TimeDelta::FromMilliseconds(200);
nqe_test_util::OverrideRTTsAndWait(rtt_2);
EXPECT_EQ(rtt_2, nqe_observer.http_rtt());
+ EXPECT_EQ(rtt_2, nqe_service->GetHttpRTT());
+ EXPECT_EQ(rtt_2, nqe_service->GetTransportRTT());
+ EXPECT_FALSE(nqe_service->GetDownstreamThroughputKbps().has_value());
+
nqe_service->RemoveRTTAndThroughputEstimatesObserver(&nqe_observer);
base::TimeDelta rtt_3 = base::TimeDelta::FromMilliseconds(300);
@@ -269,6 +277,10 @@ IN_PROC_BROWSER_TEST_F(UINetworkQualityEstimatorServiceBrowserTest,
nqe_test_util::OverrideRTTsAndWait(rtt_3);
EXPECT_EQ(rtt_2, nqe_observer.http_rtt());
+ EXPECT_EQ(rtt_3, nqe_service->GetHttpRTT());
+ EXPECT_EQ(rtt_3, nqe_service->GetTransportRTT());
+ EXPECT_FALSE(nqe_service->GetDownstreamThroughputKbps().has_value());
+
// Observer should be notified on addition.
TestRTTAndThroughputEstimatesObserver nqe_observer_2;
nqe_service->AddRTTAndThroughputEstimatesObserver(&nqe_observer_2);
« no previous file with comments | « chrome/browser/net/nqe/ui_network_quality_estimator_service.cc ('k') | net/nqe/network_quality_estimator.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698