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

Unified Diff: chrome/browser/android/data_usage/external_data_use_reporter.h

Issue 2169773002: Batch data use objects in ExternalDataUseObserver (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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: chrome/browser/android/data_usage/external_data_use_reporter.h
diff --git a/chrome/browser/android/data_usage/external_data_use_reporter.h b/chrome/browser/android/data_usage/external_data_use_reporter.h
index 5999050c89dbc2dbb246fc72fa357eaecd98399c..4231224dc1812b8df847bd6782d2fb99c0c454d6 100644
--- a/chrome/browser/android/data_usage/external_data_use_reporter.h
+++ b/chrome/browser/android/data_usage/external_data_use_reporter.h
@@ -8,6 +8,7 @@
#include <stddef.h>
#include <stdint.h>
+#include <deque>
#include <memory>
#include <string>
@@ -70,7 +71,8 @@ class ExternalDataUseReporter {
// Notifies the ExternalDataUseReporter of data usage. The data use is
// labeled using |data_use_tab_model_|, buffered and then reported to
// |external_data_use_observer_bridge_| later.
- void OnDataUse(const data_usage::DataUse& data_use);
+ void OnDataUse(
+ std::unique_ptr<std::deque<const data_usage::DataUse>> data_use_list);
tbansal1 2016/07/21 04:40:30 you can also mark the deque as const which will pr
Raj 2016/07/21 05:58:22 Done.
void OnReportDataUseDone(bool success);

Powered by Google App Engine
This is Rietveld 408576698