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

Unified Diff: components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats.cc

Issue 2575323002: Store data reduction proxy server in a separate class (Closed)
Patch Set: ryansturm comments Created 4 years 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 | components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats.cc
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats.cc
index d80fb708d53c44c794b5affd3879a76affffd60d..3aa6b85d42d790a99c0a7436f4bad587d4f719cd 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats.cc
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats.cc
@@ -20,12 +20,6 @@
#include "net/url_request/url_request.h"
#include "net/url_request/url_request_context.h"
-using net::HostPortPair;
-using net::ProxyServer;
-using net::ProxyService;
-using net::NetworkChangeNotifier;
-using net::URLRequest;
-
namespace data_reduction_proxy {
namespace {
@@ -63,7 +57,7 @@ ProxyScheme ConvertNetProxySchemeToProxyScheme(
// proxy (|is_primary| is true) or the data reduction proxy fallback.
void RecordDataReductionProxyBypassOnNetworkError(
bool is_primary,
- const ProxyServer& proxy_server,
+ const net::ProxyServer& proxy_server,
int net_error) {
if (is_primary) {
UMA_HISTOGRAM_SPARSE_SLOWLY(
@@ -139,12 +133,12 @@ DataReductionProxyBypassStats::DataReductionProxyBypassStats(
}
DataReductionProxyBypassStats::~DataReductionProxyBypassStats() {
- NetworkChangeNotifier::RemoveNetworkChangeObserver(this);
+ net::NetworkChangeNotifier::RemoveNetworkChangeObserver(this);
}
void DataReductionProxyBypassStats::InitializeOnIOThread() {
DCHECK(thread_checker_.CalledOnValidThread());
- NetworkChangeNotifier::AddNetworkChangeObserver(this);
+ net::NetworkChangeNotifier::AddNetworkChangeObserver(this);
}
void DataReductionProxyBypassStats::OnUrlRequestCompleted(
@@ -383,7 +377,7 @@ void DataReductionProxyBypassStats::RecordBypassedBytesHistograms(
}
void DataReductionProxyBypassStats::RecordMissingViaHeaderBytes(
- const URLRequest& request) {
+ const net::URLRequest& request) {
DCHECK(thread_checker_.CalledOnValidThread());
// Responses that were served from cache should have been filtered out
// already.
@@ -409,7 +403,7 @@ void DataReductionProxyBypassStats::RecordMissingViaHeaderBytes(
}
void DataReductionProxyBypassStats::OnNetworkChanged(
- NetworkChangeNotifier::ConnectionType type) {
+ net::NetworkChangeNotifier::ConnectionType type) {
DCHECK(thread_checker_.CalledOnValidThread());
ClearRequestCounts();
}
« no previous file with comments | « no previous file | components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698