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

Unified Diff: net/reporting/reporting_garbage_collector.cc

Issue 2740833004: Reporting: Implement garbage collector. (Closed)
Patch Set: Make requested changes. Created 3 years, 9 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: net/reporting/reporting_garbage_collector.cc
diff --git a/net/reporting/reporting_garbage_collector.cc b/net/reporting/reporting_garbage_collector.cc
new file mode 100644
index 0000000000000000000000000000000000000000..42e21c1bee6c94c0f23f923764d7cfbe8e00f703
--- /dev/null
+++ b/net/reporting/reporting_garbage_collector.cc
@@ -0,0 +1,41 @@
+// Copyright 2017 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.
+
+#include "net/reporting/reporting_garbage_collector.h"
+
+#include <vector>
+
+#include "base/macros.h"
+#include "base/time/time.h"
+#include "net/reporting/reporting_cache.h"
+#include "net/reporting/reporting_report.h"
+
+namespace net {
+
+// static
+void ReportingGarbageCollector::CollectGarbage(ReportingCache* cache,
+ const Policy& policy,
+ base::TimeTicks now,
+ bool network_changed) {
+ DCHECK(cache);
+
+ if (network_changed && !policy.report_persist_across_network_changes) {
+ cache->RemoveAllReports();
+ return;
+ }
+
+ std::vector<const ReportingReport*> all_reports;
+ cache->GetReports(&all_reports);
+
+ std::vector<const ReportingReport*> reports_to_remove;
+ for (const ReportingReport* report : all_reports) {
+ if (now - report->queued >= policy.report_max_age ||
+ report->attempts >= policy.report_max_attempts) {
+ reports_to_remove.push_back(report);
+ }
+ }
+ cache->RemoveReports(reports_to_remove);
+}
+
+} // namespace net

Powered by Google App Engine
This is Rietveld 408576698