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

Unified Diff: components/data_reduction_proxy/core/common/data_savings_recorder.h

Issue 2257533003: Adding a previews object, PreviewsDataSavings (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: bengr comment Created 4 years, 3 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
« no previous file with comments | « components/data_reduction_proxy/core/common/data_saver_status.h ('k') | components/previews/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/data_reduction_proxy/core/common/data_savings_recorder.h
diff --git a/components/data_reduction_proxy/core/common/data_savings_recorder.h b/components/data_reduction_proxy/core/common/data_savings_recorder.h
index 461f4173a98214c438e0c3aaa60e913a014eb7ee..a96cf5984bec230034cc60068667834cd3569bc6 100644
--- a/components/data_reduction_proxy/core/common/data_savings_recorder.h
+++ b/components/data_reduction_proxy/core/common/data_savings_recorder.h
@@ -12,18 +12,19 @@
namespace data_reduction_proxy {
// An interface that will record data savings based on the amount of data used,
// the original size of the data, and the request type.
class DataSavingsRecorder {
public:
// Records detailed data usage. Assumes |data_used| is recorded already by
// previous handling of URLRequests. Also records daily data savings
// statistics to prefs and reports data savings UMA. |data_used| and
// |original_size| are measured in bytes.
- virtual void UpdateDataSavings(const std::string& data_usage_host,
+ // Returns true if data savings was recorded.
+ virtual bool UpdateDataSavings(const std::string& data_usage_host,
int64_t data_used,
int64_t original_size) = 0;
};
} // namespace data_reduction_proxy
#endif // COMPONENTS_DATA_REDUCTION_PROXY_CORE_COMMON_DATA_SAVINGS_RECORDER_H_
« no previous file with comments | « components/data_reduction_proxy/core/common/data_saver_status.h ('k') | components/previews/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698