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

Unified Diff: net/reporting/reporting_policy.cc

Issue 2847813002: Reporting: Cap number of reports in cache. (Closed)
Patch Set: Make requested change. Created 3 years, 7 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 | « net/reporting/reporting_policy.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/reporting/reporting_policy.cc
diff --git a/net/reporting/reporting_policy.cc b/net/reporting/reporting_policy.cc
index e439fd7bfc30e43e212e2af892d715b96bde3b87..11f58fd087fa24ee9afded18c4b9115556678a9a 100644
--- a/net/reporting/reporting_policy.cc
+++ b/net/reporting/reporting_policy.cc
@@ -9,7 +9,8 @@
namespace net {
ReportingPolicy::ReportingPolicy()
- : delivery_interval(base::TimeDelta::FromMinutes(1)),
+ : max_report_count(100u),
+ delivery_interval(base::TimeDelta::FromMinutes(1)),
persistence_interval(base::TimeDelta::FromMinutes(1)),
persist_reports_across_restarts(false),
persist_clients_across_restarts(true),
@@ -28,7 +29,8 @@ ReportingPolicy::ReportingPolicy()
}
ReportingPolicy::ReportingPolicy(const ReportingPolicy& other)
- : delivery_interval(base::TimeDelta::FromMinutes(1)),
+ : max_report_count(other.max_report_count),
+ delivery_interval(other.delivery_interval),
endpoint_backoff_policy(other.endpoint_backoff_policy),
persistence_interval(other.persistence_interval),
persist_reports_across_restarts(other.persist_reports_across_restarts),
« no previous file with comments | « net/reporting/reporting_policy.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698