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

Unified Diff: components/data_reduction_proxy/core/browser/data_reduction_proxy_config.h

Issue 1140443002: Modify IPC call to properly record the PLT histograms for LoFi. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Minor fixes to comments. Created 5 years, 7 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
Index: components/data_reduction_proxy/core/browser/data_reduction_proxy_config.h
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_config.h b/components/data_reduction_proxy/core/browser/data_reduction_proxy_config.h
index 64f75897543b26c1eae903f3b253e1ef41fcdad4..c0083cb1dbfe0e36040a9e2096cbd84a1b871ac8 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_config.h
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_config.h
@@ -186,6 +186,19 @@ class DataReductionProxyConfig
// tied to whether the Data Reduction Proxy is enabled.
bool promo_allowed() const;
+ // Returns true if current network conditions are worse than the ones
+ // specified in the enabled or control field trial group parameters.
+ // Virtualized for mocking.
+ virtual bool IsNetworkBad() const;
+
+ // Returns true if this client is part of LoFi enabled field trial.
+ // Virtualized for mocking.
+ virtual bool IsIncludedInLoFiEnabledFieldTrial() const;
+
+ // Returns true if this client is part of LoFi control field trial.
+ // Virtualized for mocking.
+ virtual bool IsIncludedInLoFiControlFieldTrial() const;
+
protected:
// Writes a warning to the log that is used in backend processing of
// customer feedback. Virtual so tests can mock it for verification.

Powered by Google App Engine
This is Rietveld 408576698