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

Unified Diff: chrome/browser/net/nqe/ui_network_quality_estimator_service.h

Issue 2742293004: Expose getters methods for NQE on UI thread (Closed)
Patch Set: ps 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 | « no previous file | chrome/browser/net/nqe/ui_network_quality_estimator_service.cc » ('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.h
diff --git a/chrome/browser/net/nqe/ui_network_quality_estimator_service.h b/chrome/browser/net/nqe/ui_network_quality_estimator_service.h
index af77188b0b2f105890a74f46b6bd06f955d14efa..7190140334c134831f20afd761f07abaf0b02155 100644
--- a/chrome/browser/net/nqe/ui_network_quality_estimator_service.h
+++ b/chrome/browser/net/nqe/ui_network_quality_estimator_service.h
@@ -46,6 +46,11 @@ class UINetworkQualityEstimatorService
void RemoveEffectiveConnectionTypeObserver(
net::NetworkQualityEstimator::EffectiveConnectionTypeObserver* observer)
override;
+ bool GetHttpRTT(base::TimeDelta* http_rtt) const override WARN_UNUSED_RESULT;
RyanSturm 2017/03/15 18:25:30 Return base::optional<TYPE> instead for these?
tbansal1 2017/03/15 22:34:24 Done.
+ bool GetTransportRTT(base::TimeDelta* transport_rtt) const override
+ WARN_UNUSED_RESULT;
+ bool GetDownstreamThroughputKbps(
+ int32_t* downstream_throughput_kbps) const override WARN_UNUSED_RESULT;
RyanSturm 2017/03/15 18:25:30 include <stdint.h>
tbansal1 2017/03/15 22:34:23 Done.
// Must be called on the UI thread. |observer| will be notified on the UI
// thread. |observer| would be notified of the changes in the HTTP RTT,
« no previous file with comments | « no previous file | chrome/browser/net/nqe/ui_network_quality_estimator_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698