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

Unified Diff: chrome/browser/net/chrome_network_delegate.h

Issue 775773002: Add data reduction proxy debug info to net-internals#bandwidth (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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
Index: chrome/browser/net/chrome_network_delegate.h
diff --git a/chrome/browser/net/chrome_network_delegate.h b/chrome/browser/net/chrome_network_delegate.h
index ed0a939f27fa9be37ba4c2d03ebbb23c653a03a7..a4d09b9d3b869c2e9b8d572677f7e28b2729d9a6 100644
--- a/chrome/browser/net/chrome_network_delegate.h
+++ b/chrome/browser/net/chrome_network_delegate.h
@@ -14,6 +14,7 @@
#include "base/memory/scoped_ptr.h"
#include "base/values.h"
#include "components/data_reduction_proxy/core/browser/data_reduction_proxy_metrics.h"
+#include "components/data_reduction_proxy/core/common/data_reduction_proxy_event_store.h"
#include "net/base/network_delegate.h"
#include "net/proxy/proxy_retry_info.h"
@@ -37,6 +38,7 @@ class Predictor;
namespace data_reduction_proxy {
class DataReductionProxyAuthRequestHandler;
+class DataReductionProxyEventStore;
bengr 2014/12/02 23:55:32 Please coordinate with megjablon@, who is refactor
jeremyim 2014/12/03 08:10:36 Acknowledged.
class DataReductionProxyParams;
class DataReductionProxyStatisticsPrefs;
class DataReductionProxyUsageStats;
@@ -188,6 +190,14 @@ class ChromeNetworkDelegate : public net::NetworkDelegate {
data_reduction_proxy_statistics_prefs_ = statistics_prefs;
}
+ // |data_reduction_proxy_event_store_| must outlive this
+ // ChromeNetworkDelegate.
+ void set_data_reduction_proxy_event_store(
+ data_reduction_proxy::DataReductionProxyEventStore*
+ event_store) {
bengr 2014/12/02 23:55:32 Move to previous line.
jeremyim 2014/12/03 08:10:36 Done.
+ data_reduction_proxy_event_store_ = event_store;
+ }
+
void set_on_resolve_proxy_handler(OnResolveProxyHandler handler) {
on_resolve_proxy_handler_ = handler;
}
@@ -227,6 +237,10 @@ class ChromeNetworkDelegate : public net::NetworkDelegate {
// responsible for deleting the returned value.
base::Value* SessionNetworkStatsInfoToValue() const;
+ // Creates a Value summary of data reduction proxy related information. The
bengr 2014/12/02 23:55:32 Please detail what is included in this summary.
jeremyim 2014/12/03 08:10:36 Done.
+ // caller is responsible for deleting the returned value.
+ base::Value* DataReductionProxyInfoToValue() const;
+
private:
friend class ChromeNetworkDelegateTest;
@@ -343,6 +357,10 @@ class ChromeNetworkDelegate : public net::NetworkDelegate {
data_reduction_proxy_auth_request_handler_;
data_reduction_proxy::DataReductionProxyStatisticsPrefs*
data_reduction_proxy_statistics_prefs_;
+ // |data_reduction_proxy_event_store_| must outlive this
+ // ChromeNetworkDelegate.
+ data_reduction_proxy::DataReductionProxyEventStore*
+ data_reduction_proxy_event_store_;
OnResolveProxyHandler on_resolve_proxy_handler_;
ProxyConfigGetter proxy_config_getter_;
« no previous file with comments | « no previous file | chrome/browser/net/chrome_network_delegate.cc » ('j') | chrome/browser/net/chrome_network_delegate.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698