Implement important sites dialog for desktop.

chrome/browser/browsing_data/browsing_data_important_sites_util.cc

Index: chrome/browser/browsing_data/browsing_data_important_sites_util.cc
diff --git a/chrome/browser/browsing_data/browsing_data_important_sites_util.cc b/chrome/browser/browsing_data/browsing_data_important_sites_util.cc
new file mode 100644
index 0000000000000000000000000000000000000000..799acfbe39c70ced11b924fc9d26567a1bfd6a8d
--- /dev/null
+++ b/chrome/browser/browsing_data/browsing_data_important_sites_util.cc
@@ -0,0 +1,47 @@
+// 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 "chrome/browser/browsing_data/browsing_data_important_sites_util.h"
+
+#include "content/public/browser/browsing_data_filter_builder.h"
+
+namespace browsing_data_important_sites_util {
+
+void Remove(int remove_mask,
+            int origin_mask,
+            browsing_data::TimePeriod time_period,
+            std::unique_ptr<content::BrowsingDataFilterBuilder> filter_builder,
+            BrowsingDataRemover* remover,
+            BrowsingDataRemover::Observer* observer) {
+  int filterable_mask =
+      remove_mask &
+      ChromeBrowsingDataRemoverDelegate::IMPORTANT_SITES_DATA_TYPES;
+  int nonfilterable_mask =
+      remove_mask &
+      ~ChromeBrowsingDataRemoverDelegate::IMPORTANT_SITES_DATA_TYPES;
+
+  browsing_data::RecordDeletionForPeriod(time_period);
+
+  if (filterable_mask) {
+    remover->RemoveWithFilterAndReply(
+        browsing_data::CalculateBeginDeleteTime(time_period),
+        browsing_data::CalculateEndDeleteTime(time_period), filterable_mask,
+        origin_mask, std::move(filter_builder), observer);
+  } else {
+    // Make sure |observer| doesn't wait for the filtered task.

[dschuyler, 2017/04/07 00:02:05]

Nit: the comment here and on line 42 seem unnecessary. i.e. it seems evident
from being in the else block of this if() statement.
(If the comments are trying to convey something subtle/important then my
feedback is that I'm not getting that subtle/important tip from the current
wording).

[dullweber, 2017/04/07 09:39:21]

Done.

+    observer->OnBrowsingDataRemoverDone();
+  }
+
+  if (nonfilterable_mask) {
+    remover->RemoveAndReply(
+        browsing_data::CalculateBeginDeleteTime(time_period),
+        browsing_data::CalculateEndDeleteTime(time_period), nonfilterable_mask,
+        origin_mask, observer);
+  } else {
+    // Make sure |observer| doesn't wait for the non-filtered task.
+    observer->OnBrowsingDataRemoverDone();
+  }
+}
+
+}  // namespace browsing_data_important_sites_util