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

Unified Diff: components/data_reduction_proxy/browser/data_reduction_proxy_settings.h

Issue 412143009: Moved data reduction proxy initialization logic to ProfileImplIOData (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments from willchan Created 6 years, 5 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/browser/data_reduction_proxy_settings.h
diff --git a/components/data_reduction_proxy/browser/data_reduction_proxy_settings.h b/components/data_reduction_proxy/browser/data_reduction_proxy_settings.h
index 59b215738900863b0b725b45f6d26463ac4bbe16..0346940a46b04498007673c8042ef63609ccef2a 100644
--- a/components/data_reduction_proxy/browser/data_reduction_proxy_settings.h
+++ b/components/data_reduction_proxy/browser/data_reduction_proxy_settings.h
@@ -15,7 +15,6 @@
#include "base/threading/thread_checker.h"
#include "components/data_reduction_proxy/browser/data_reduction_proxy_configurator.h"
#include "components/data_reduction_proxy/browser/data_reduction_proxy_params.h"
-#include "components/data_reduction_proxy/browser/data_reduction_proxy_usage_stats.h"
#include "net/base/net_util.h"
#include "net/base/network_change_notifier.h"
#include "net/url_request/url_fetcher_delegate.h"
@@ -97,10 +96,6 @@ class DataReductionProxySettings
return params_.get();
}
- DataReductionProxyUsageStats* usage_stats() const {
- return usage_stats_;
- }
-
// Initializes the data reduction proxy with profile and local state prefs,
// and a |UrlRequestContextGetter| for canary probes. The caller must ensure
// that all parameters remain alive for the lifetime of the
@@ -159,15 +154,14 @@ class DataReductionProxySettings
int64* received_content_length,
int64* last_update_time);
+ // Records that the data reduction proxy is unreachable or not.
+ void SetUnreachable(bool unreachable);
+
// Returns whether the data reduction proxy is unreachable. Returns true
// if no request has successfully completed through proxy, even though atleast
// some of them should have.
bool IsDataReductionProxyUnreachable();
- // Set the data reduction proxy usage stats.
- void SetDataReductionProxyUsageStats(
- DataReductionProxyUsageStats* usage_stats);
-
// Returns an vector containing the aggregate received HTTP content in the
// last |kNumDaysInHistory| days.
ContentLengthList GetDailyReceivedContentLengths();
@@ -298,6 +292,7 @@ class DataReductionProxySettings
bool restricted_by_carrier_;
bool enabled_by_user_;
bool disabled_on_vpn_;
+ bool unreachable_;
scoped_ptr<net::URLFetcher> fetcher_;
scoped_ptr<net::URLFetcher> warmup_fetcher_;
@@ -315,7 +310,6 @@ class DataReductionProxySettings
base::ThreadChecker thread_checker_;
scoped_ptr<DataReductionProxyParams> params_;
- DataReductionProxyUsageStats* usage_stats_;
DISALLOW_COPY_AND_ASSIGN(DataReductionProxySettings);
};

Powered by Google App Engine
This is Rietveld 408576698