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

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

Issue 2733393003: Split browsing data masks between content and embedder (Closed)
Patch Set: Rebase (merged automatically) 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
1 // Copyright 2017 The Chromium Authors. All rights reserved. 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 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/browsing_data/mock_browsing_data_remover.h" 5 #include "chrome/browser/browsing_data/mock_browsing_data_remover.h"
6 6
7 #include "content/public/browser/browsing_data_filter_builder.h" 7 #include "content/public/browser/browsing_data_filter_builder.h"
8 #include "testing/gtest/include/gtest/gtest.h" 8 #include "testing/gtest/include/gtest/gtest.h"
9 9
10 namespace { 10 namespace {
(...skipping 22 matching lines...) Expand all
33 const base::Time& delete_end, 33 const base::Time& delete_end,
34 int remove_mask, 34 int remove_mask,
35 int origin_type_mask, 35 int origin_type_mask,
36 Observer* observer) { 36 Observer* observer) {
37 DCHECK(observer); 37 DCHECK(observer);
38 RemoveInternal(delete_begin, delete_end, remove_mask, origin_type_mask, 38 RemoveInternal(delete_begin, delete_end, remove_mask, origin_type_mask,
39 std::unique_ptr<content::BrowsingDataFilterBuilder>(), 39 std::unique_ptr<content::BrowsingDataFilterBuilder>(),
40 observer); 40 observer);
41 } 41 }
42 42
43 bool MockBrowsingDataRemover::DoesOriginMatchMask(
44 int origin_type_mask,
45 const GURL& origin,
46 storage::SpecialStoragePolicy* special_storage_policy) const {
47 return true;
48 }
49
43 void MockBrowsingDataRemover::RemoveWithFilter( 50 void MockBrowsingDataRemover::RemoveWithFilter(
44 const base::Time& delete_begin, 51 const base::Time& delete_begin,
45 const base::Time& delete_end, 52 const base::Time& delete_end,
46 int remove_mask, 53 int remove_mask,
47 int origin_type_mask, 54 int origin_type_mask,
48 std::unique_ptr<content::BrowsingDataFilterBuilder> filter_builder) { 55 std::unique_ptr<content::BrowsingDataFilterBuilder> filter_builder) {
49 DCHECK_EQ(0, remove_mask & ~FILTERABLE_DATATYPES);
50 DCHECK(filter_builder); 56 DCHECK(filter_builder);
51 RemoveInternal(delete_begin, delete_end, remove_mask, origin_type_mask, 57 RemoveInternal(delete_begin, delete_end, remove_mask, origin_type_mask,
52 std::move(filter_builder), nullptr); 58 std::move(filter_builder), nullptr);
53 } 59 }
54 60
55 void MockBrowsingDataRemover::RemoveWithFilterAndReply( 61 void MockBrowsingDataRemover::RemoveWithFilterAndReply(
56 const base::Time& delete_begin, 62 const base::Time& delete_begin,
57 const base::Time& delete_end, 63 const base::Time& delete_end,
58 int remove_mask, 64 int remove_mask,
59 int origin_type_mask, 65 int origin_type_mask,
60 std::unique_ptr<content::BrowsingDataFilterBuilder> filter_builder, 66 std::unique_ptr<content::BrowsingDataFilterBuilder> filter_builder,
61 Observer* observer) { 67 Observer* observer) {
62 DCHECK_EQ(0, remove_mask & ~FILTERABLE_DATATYPES);
63 DCHECK(filter_builder); 68 DCHECK(filter_builder);
64 DCHECK(observer); 69 DCHECK(observer);
65 RemoveInternal(delete_begin, delete_end, remove_mask, origin_type_mask, 70 RemoveInternal(delete_begin, delete_end, remove_mask, origin_type_mask,
66 std::move(filter_builder), observer); 71 std::move(filter_builder), observer);
67 } 72 }
68 73
69 void MockBrowsingDataRemover::RemoveInternal( 74 void MockBrowsingDataRemover::RemoveInternal(
70 const base::Time& delete_begin, 75 const base::Time& delete_begin,
71 const base::Time& delete_end, 76 const base::Time& delete_end,
72 int remove_mask, 77 int remove_mask,
(...skipping 99 matching lines...) Expand 10 before | Expand all | Expand 10 after
172 if (a.delete_begin_ != b.delete_begin_ || a.delete_end_ != b.delete_end_ || 177 if (a.delete_begin_ != b.delete_begin_ || a.delete_end_ != b.delete_end_ ||
173 a.remove_mask_ != b.remove_mask_ || 178 a.remove_mask_ != b.remove_mask_ ||
174 a.origin_type_mask_ != b.origin_type_mask_) { 179 a.origin_type_mask_ != b.origin_type_mask_) {
175 return false; 180 return false;
176 } 181 }
177 182
178 if (!a.should_compare_filter_ || !b.should_compare_filter_) 183 if (!a.should_compare_filter_ || !b.should_compare_filter_)
179 return true; 184 return true;
180 return *a.filter_builder_ == *b.filter_builder_; 185 return *a.filter_builder_ == *b.filter_builder_;
181 } 186 }
OLDNEW
« no previous file with comments | « chrome/browser/browsing_data/mock_browsing_data_remover.h ('k') | chrome/browser/chrome_content_browser_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698