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

Unified Diff: components/data_usage/core/data_use_aggregator.h

Issue 1373373002: Create component to expose network usage stats to consumers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments Created 5 years, 2 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_usage/core/data_use_aggregator.h
diff --git a/components/data_usage/core/data_use_aggregator.h b/components/data_usage/core/data_use_aggregator.h
new file mode 100644
index 0000000000000000000000000000000000000000..4c14ec1c3d3b38273096d94fc940b563d2ef1834
--- /dev/null
+++ b/components/data_usage/core/data_use_aggregator.h
@@ -0,0 +1,64 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_DATA_USAGE_CORE_DATA_USE_AGGREGATOR_H_
+#define COMPONENTS_DATA_USAGE_CORE_DATA_USE_AGGREGATOR_H_
+
+#include <stdint.h>
+
+#include <vector>
+
+#include "base/macros.h"
+#include "base/observer_list.h"
+#include "base/threading/thread_checker.h"
+#include "components/data_usage/core/data_use.h"
+
+namespace net {
+class URLRequest;
+}
+
+namespace data_usage {
+
+// Class that collects and aggregates network usage, reporting the usage to
+// observers. Should only be used on the IO thread.
+class DataUseAggregator {
+ public:
+ class Observer {
+ public:
+ virtual ~Observer() {}
+ // TODO(sclittle): Consider performing some pre-aggregation on the data use.
+ virtual void OnDataUse(const std::vector<DataUse>& data_use_sequence) = 0;
tbansal1 2015/10/08 04:22:44 DISALLOW_COPY_AND_ASSIGN?
sclittle 2015/10/08 19:44:13 The subclasses should DISALLOW_COPY_AND_ASSIGN the
+ };
+
+ DataUseAggregator();
+ virtual ~DataUseAggregator();
+
+ void AddObserver(Observer* observer);
+ void RemoveObserver(Observer* observer);
+
+ // Virtual for testing.
+ virtual void ReportDataUse(const net::URLRequest& request,
+ int32_t tab_id,
+ int64_t tx_bytes,
+ int64_t rx_bytes);
+
+ // Account for off-the-record data use. This usage is only kept track of here
+ // so that it can be taken out of any amortized data usage calculations, and a
+ // per-request breakdown of off-the-record data usage will never leave the
+ // DataUseAggregator.
+ // Virtual for testing.
+ virtual void ReportOffTheRecordDataUse(int64_t tx_bytes, int64_t rx_bytes);
+
+ private:
+ void NotifyDataUse(const std::vector<DataUse>& data_use_sequence);
+
+ base::ThreadChecker thread_checker_;
+ base::ObserverList<Observer> observer_list_;
+
+ DISALLOW_COPY_AND_ASSIGN(DataUseAggregator);
+};
+
+} // namespace data_usage
+
+#endif // COMPONENTS_DATA_USAGE_CORE_DATA_USE_AGGREGATOR_H_

Powered by Google App Engine
This is Rietveld 408576698