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

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

Issue 1074293003: Use int64 instead of int32 for response sizes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Reuploading to try to fix corrupt issue Created 5 years, 8 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_reduction_proxy/core/browser/data_reduction_proxy_compression_stats.cc
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_compression_stats.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_compression_stats.cc
index badf9531fa0ab732174a3783c0ac33018db9c382..5655e12c92fdd4fc8fe6f62922396b85505649df 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_compression_stats.cc
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_compression_stats.cc
@@ -216,7 +216,7 @@ class DailyContentLengthUpdate {
}
// Update the lengths for the current day.
- void Add(int content_length) {
+ void Add(int64 content_length) {
AddInt64ToListPref(kNumDaysInHistory - 1, content_length, update_);
}
@@ -280,7 +280,7 @@ class DailyDataSavingUpdate {
}
// Update the lengths for the current day.
- void Add(int original_content_length, int received_content_length) {
+ void Add(int64 original_content_length, int64 received_content_length) {
original_.Add(original_content_length);
received_.Add(received_content_length);
}
@@ -367,8 +367,8 @@ void DataReductionProxyCompressionStats::OnUpdateContentLengths() {
}
void DataReductionProxyCompressionStats::UpdateContentLengths(
- int received_content_length,
- int original_content_length,
+ int64 received_content_length,
+ int64 original_content_length,
bool data_reduction_proxy_enabled,
DataReductionProxyRequestType request_type) {
DCHECK(thread_checker_.CalledOnValidThread());
@@ -594,11 +594,11 @@ int64 DataReductionProxyCompressionStats::GetListPrefInt64Value(
}
void DataReductionProxyCompressionStats::RecordContentLengthPrefs(
- int received_content_length,
- int original_content_length,
- bool with_data_reduction_proxy_enabled,
- DataReductionProxyRequestType request_type,
- base::Time now) {
+ int64 received_content_length,
+ int64 original_content_length,
+ bool with_data_reduction_proxy_enabled,
+ DataReductionProxyRequestType request_type,
+ base::Time now) {
// TODO(bengr): Remove this check once the underlying cause of
// http://crbug.com/287821 is fixed. For now, only continue if the current
// year is reported as being between 1972 and 2970.

Powered by Google App Engine
This is Rietveld 408576698