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

Side by Side Diff: chrome/browser/browsing_data/browsing_data_important_sites_util.cc

Issue 2716333002: Implement important sites dialog for desktop. (Closed)
Patch Set: log histogram when important sites is shown 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/browser/browsing_data/browsing_data_important_sites_util.h"
6
7 #include "content/public/browser/browsing_data_filter_builder.h"
8
9 namespace browsing_data_important_sites_util {
10
11 void Remove(int remove_mask,
12 int origin_mask,
13 browsing_data::TimePeriod time_period,
14 std::unique_ptr<content::BrowsingDataFilterBuilder> filter_builder,
15 BrowsingDataRemover* remover,
16 BrowsingDataRemover::Observer* observer) {
17 int filterable_mask =
18 remove_mask & BrowsingDataRemover::IMPORTANT_SITES_DATATYPES;
19 int nonfilterable_mask =
20 remove_mask & ~BrowsingDataRemover::IMPORTANT_SITES_DATATYPES;
21
22 browsing_data::RecordDeletionForPeriod(time_period);
23
24 if (filterable_mask) {
25 remover->RemoveWithFilterAndReply(
26 browsing_data::CalculateBeginDeleteTime(time_period),
27 browsing_data::CalculateEndDeleteTime(time_period), filterable_mask,
28 origin_mask, std::move(filter_builder), observer);
29 } else {
30 // Make sure |observer| doesn't wait for the filtered task.
31 observer->OnBrowsingDataRemoverDone();
32 }
33
34 if (nonfilterable_mask) {
35 remover->RemoveAndReply(
36 browsing_data::CalculateBeginDeleteTime(time_period),
37 browsing_data::CalculateEndDeleteTime(time_period), nonfilterable_mask,
38 origin_mask, observer);
39 } else {
40 // Make sure |observer| doesn't wait for the non-filtered task.
41 observer->OnBrowsingDataRemoverDone();
42 }
43 }
44
45 } // namespace browsing_data_important_sites_util
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698