Reporting: Plumb from //chrome/browser/browsing_data.

net/reporting/reporting_browsing_data_remover.cc

 // 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_browsing_data_remover.h"
 
 #include <vector>
 
+#include "base/memory/ptr_util.h"
 #include "net/reporting/reporting_cache.h"
+#include "net/reporting/reporting_client.h"
 #include "net/reporting/reporting_context.h"
 #include "net/reporting/reporting_report.h"
 
 namespace net {
 
-// static
-void ReportingBrowsingDataRemover::RemoveBrowsingData(
-    ReportingContext* context,
-    int data_type_mask,
-    base::Callback<bool(const GURL&)> origin_filter) {
-  ReportingCache* cache = context->cache();
-  bool remove_reports = (data_type_mask & DATA_TYPE_REPORTS) != 0;
-  bool remove_clients = (data_type_mask & DATA_TYPE_CLIENTS) != 0;
+namespace {
 
-  if (origin_filter.is_null()) {
-    if (remove_reports)
-      cache->RemoveAllReports();
-    if (remove_clients)
-      cache->RemoveAllClients();
-    return;
+class ReportingBrowsingDataRemoverImpl : public ReportingBrowsingDataRemover {
+ public:
+  ReportingBrowsingDataRemoverImpl(ReportingContext* context)
+      : context_(context) {}
+
+  // ReportingBrowsingDataRemover implementation:
+
+  ~ReportingBrowsingDataRemoverImpl() override {}
+
+  void RemoveBrowsingData(
+      int data_type_mask,
+      base::Callback<bool(const GURL&)> origin_filter) override {
+    bool remove_reports = (data_type_mask & DATA_TYPE_REPORTS) != 0;
+    bool remove_clients = (data_type_mask & DATA_TYPE_CLIENTS) != 0;
+

[shivanisha, 2017/04/20 18:21:52]

ReportingCache* cache = context_->cache(); instead of invoking cache() every
time and then cache() can also be removed.

[Julia Tuttle, 2017/04/21 13:34:53]

Done.

+    if (origin_filter.is_null()) {
+      if (remove_reports)
+        cache()->RemoveAllReports();
+      if (remove_clients)
+        cache()->RemoveAllClients();
+      return;
+    }
+
+    if (remove_reports) {
+      std::vector<const ReportingReport*> all_reports;
+      cache()->GetReports(&all_reports);
+
+      std::vector<const ReportingReport*> reports_to_remove;
+      for (const ReportingReport* report : all_reports) {
+        if (origin_filter.Run(report->url))
+          reports_to_remove.push_back(report);
+      }
+
+      cache()->RemoveReports(reports_to_remove);
+    }
+
+    if (remove_clients) {
+      std::vector<const ReportingClient*> all_clients;
+      cache()->GetClients(&all_clients);
+
+      std::vector<const ReportingClient*> clients_to_remove;
+      for (const ReportingClient* client : all_clients) {
+        // TODO(juliatuttle): Examine client endpoint as well?
+        if (origin_filter.Run(client->origin.GetURL()))
+          clients_to_remove.push_back(client);
+      }
+
+      cache()->RemoveClients(clients_to_remove);
+    }
   }
 
-  if (remove_reports) {
-    std::vector<const ReportingReport*> all_reports;
-    cache->GetReports(&all_reports);
+ private:
+  ReportingCache* cache() { return context_->cache(); }
 
-    std::vector<const ReportingReport*> reports_to_remove;
-    for (const ReportingReport* report : all_reports) {
-      if (origin_filter.Run(report->url))
-        reports_to_remove.push_back(report);
-    }
+  ReportingContext* context_;
 
-    cache->RemoveReports(reports_to_remove);
-  }
+  DISALLOW_COPY_AND_ASSIGN(ReportingBrowsingDataRemoverImpl);
+};
 
-  if (remove_clients) {
-    std::vector<const ReportingClient*> all_clients;
-    cache->GetClients(&all_clients);
+}  // namespace
 
-    std::vector<const ReportingClient*> clients_to_remove;
-    for (const ReportingClient* client : all_clients) {
-      if (origin_filter.Run(client->origin.GetURL()))
-        clients_to_remove.push_back(client);
-    }
-
-    cache->RemoveClients(clients_to_remove);
-  }
+// static
+std::unique_ptr<ReportingBrowsingDataRemover>
+ReportingBrowsingDataRemover::Create(ReportingContext* context) {
+  return base::MakeUnique<ReportingBrowsingDataRemoverImpl>(context);
 }
 
+ReportingBrowsingDataRemover::~ReportingBrowsingDataRemover() {}
+
 }  // namespace net