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

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

Issue 2768883002: Change MockBrowsingDataRemover to MockBrowsingDataRemoverDelegate (Closed)
Patch Set: Fixed the comment. 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/mock_browsing_data_remover.h"
6
7 #include "content/public/browser/browsing_data_filter_builder.h"
8 #include "testing/gtest/include/gtest/gtest.h"
9
10 namespace {
11 base::Time some_time;
12 }
13
14 MockBrowsingDataRemover::MockBrowsingDataRemover(
15 content::BrowserContext* context) {}
16
17 MockBrowsingDataRemover::~MockBrowsingDataRemover() {
18 DCHECK(!expected_calls_.size()) << "Expectations were set but not verified.";
19 }
20
21 void MockBrowsingDataRemover::Shutdown() {}
22
23 void MockBrowsingDataRemover::Remove(const base::Time& delete_begin,
24 const base::Time& delete_end,
25 int remove_mask,
26 int origin_type_mask) {
27 RemoveInternal(delete_begin, delete_end, remove_mask, origin_type_mask,
28 std::unique_ptr<content::BrowsingDataFilterBuilder>(),
29 nullptr);
30 }
31
32 void MockBrowsingDataRemover::RemoveAndReply(const base::Time& delete_begin,
33 const base::Time& delete_end,
34 int remove_mask,
35 int origin_type_mask,
36 Observer* observer) {
37 DCHECK(observer);
38 RemoveInternal(delete_begin, delete_end, remove_mask, origin_type_mask,
39 std::unique_ptr<content::BrowsingDataFilterBuilder>(),
40 observer);
41 }
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
50 void MockBrowsingDataRemover::RemoveWithFilter(
51 const base::Time& delete_begin,
52 const base::Time& delete_end,
53 int remove_mask,
54 int origin_type_mask,
55 std::unique_ptr<content::BrowsingDataFilterBuilder> filter_builder) {
56 DCHECK(filter_builder);
57 RemoveInternal(delete_begin, delete_end, remove_mask, origin_type_mask,
58 std::move(filter_builder), nullptr);
59 }
60
61 void MockBrowsingDataRemover::RemoveWithFilterAndReply(
62 const base::Time& delete_begin,
63 const base::Time& delete_end,
64 int remove_mask,
65 int origin_type_mask,
66 std::unique_ptr<content::BrowsingDataFilterBuilder> filter_builder,
67 Observer* observer) {
68 DCHECK(filter_builder);
69 DCHECK(observer);
70 RemoveInternal(delete_begin, delete_end, remove_mask, origin_type_mask,
71 std::move(filter_builder), observer);
72 }
73
74 void MockBrowsingDataRemover::RemoveInternal(
75 const base::Time& delete_begin,
76 const base::Time& delete_end,
77 int remove_mask,
78 int origin_type_mask,
79 std::unique_ptr<content::BrowsingDataFilterBuilder> filter_builder,
80 BrowsingDataRemover::Observer* observer) {
81 actual_calls_.emplace_back(delete_begin, delete_end, remove_mask,
82 origin_type_mask, std::move(filter_builder),
83 true /* should_compare_filter */);
84
85 // |observer| is not recorded in |actual_calls_| to be compared with
86 // expectations, because it's created internally in ClearSiteData() and
87 // it's unknown to this. However, it is tested implicitly, because we use
88 // it for the completion callback, so an incorrect |observer| will fail
89 // the test by waiting for the callback forever.
90 DCHECK(observer);
91 observer->OnBrowsingDataRemoverDone();
92 }
93
94 void MockBrowsingDataRemover::AddObserver(Observer* observer) {}
95
96 void MockBrowsingDataRemover::RemoveObserver(Observer* observer) {}
97
98 const base::Time& MockBrowsingDataRemover::GetLastUsedBeginTime() {
99 NOTIMPLEMENTED();
100 return some_time;
101 }
102
103 const base::Time& MockBrowsingDataRemover::GetLastUsedEndTime() {
104 NOTIMPLEMENTED();
105 return some_time;
106 }
107
108 int MockBrowsingDataRemover::GetLastUsedRemovalMask() {
109 NOTIMPLEMENTED();
110 return -1;
111 }
112
113 int MockBrowsingDataRemover::GetLastUsedOriginTypeMask() {
114 NOTIMPLEMENTED();
115 return -1;
116 }
117
118 void MockBrowsingDataRemover::SetEmbedderDelegate(
119 std::unique_ptr<BrowsingDataRemoverDelegate> embedder_delegate) {
120 NOTIMPLEMENTED();
121 }
122
123 BrowsingDataRemoverDelegate* MockBrowsingDataRemover::GetEmbedderDelegate()
124 const {
125 NOTIMPLEMENTED();
126 return nullptr;
127 }
128
129 void MockBrowsingDataRemover::ExpectCall(
130 const base::Time& delete_begin,
131 const base::Time& delete_end,
132 int remove_mask,
133 int origin_type_mask,
134 std::unique_ptr<content::BrowsingDataFilterBuilder> filter_builder) {
135 expected_calls_.emplace_back(delete_begin, delete_end, remove_mask,
136 origin_type_mask, std::move(filter_builder),
137 true /* should_compare_filter */);
138 }
139
140 void MockBrowsingDataRemover::ExpectCallDontCareAboutFilterBuilder(
141 const base::Time& delete_begin,
142 const base::Time& delete_end,
143 int remove_mask,
144 int origin_type_mask) {
145 expected_calls_.emplace_back(
146 delete_begin, delete_end, remove_mask, origin_type_mask,
147 std::unique_ptr<content::BrowsingDataFilterBuilder>(),
148 false /* should_compare_filter */);
149 }
150
151 void MockBrowsingDataRemover::VerifyAndClearExpectations() {
152 EXPECT_EQ(expected_calls_, actual_calls_);
153 expected_calls_.clear();
154 actual_calls_.clear();
155 }
156
157 MockBrowsingDataRemover::CallParameters::CallParameters(
158 const base::Time& delete_begin,
159 const base::Time& delete_end,
160 int remove_mask,
161 int origin_type_mask,
162 std::unique_ptr<content::BrowsingDataFilterBuilder> filter_builder,
163 bool should_compare_filter)
164 : delete_begin_(delete_begin),
165 delete_end_(delete_end),
166 remove_mask_(remove_mask),
167 origin_type_mask_(origin_type_mask),
168 filter_builder_(std::move(filter_builder)),
169 should_compare_filter_(should_compare_filter) {}
170 MockBrowsingDataRemover::CallParameters::~CallParameters() {}
171
172 bool MockBrowsingDataRemover::CallParameters::operator==(
173 const CallParameters& other) const {
174 const CallParameters& a = *this;
175 const CallParameters& b = other;
176
177 if (a.delete_begin_ != b.delete_begin_ || a.delete_end_ != b.delete_end_ ||
178 a.remove_mask_ != b.remove_mask_ ||
179 a.origin_type_mask_ != b.origin_type_mask_) {
180 return false;
181 }
182
183 if (!a.should_compare_filter_ || !b.should_compare_filter_)
184 return true;
185 return *a.filter_builder_ == *b.filter_builder_;
186 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698