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

Side by Side Diff: content/public/test/mock_browsing_data_remover_delegate.cc

Issue 2368923003: Support the Clear-Site-Data header on resource requests (Closed)
Patch Set: Addressed comments Created 3 years, 6 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_delegate.h" 5 #include "content/public/test/mock_browsing_data_remover_delegate.h"
6 6
7 #include "base/callback.h" 7 #include "base/callback.h"
8 #include "content/public/browser/browsing_data_filter_builder.h" 8 #include "content/public/browser/browsing_data_filter_builder.h"
9 #include "testing/gtest/include/gtest/gtest.h" 9 #include "testing/gtest/include/gtest/gtest.h"
10 10
11 namespace content {
12
11 MockBrowsingDataRemoverDelegate::MockBrowsingDataRemoverDelegate() {} 13 MockBrowsingDataRemoverDelegate::MockBrowsingDataRemoverDelegate() {}
12 14
13 MockBrowsingDataRemoverDelegate::~MockBrowsingDataRemoverDelegate() { 15 MockBrowsingDataRemoverDelegate::~MockBrowsingDataRemoverDelegate() {
14 DCHECK(!expected_calls_.size()) << "Expectations were set but not verified."; 16 DCHECK(!expected_calls_.size()) << "Expectations were set but not verified.";
15 } 17 }
16 18
17 content::BrowsingDataRemoverDelegate::EmbedderOriginTypeMatcher 19 content::BrowsingDataRemoverDelegate::EmbedderOriginTypeMatcher
18 MockBrowsingDataRemoverDelegate::GetOriginTypeMatcher() const { 20 MockBrowsingDataRemoverDelegate::GetOriginTypeMatcher() const {
19 return content::BrowsingDataRemoverDelegate::EmbedderOriginTypeMatcher(); 21 return content::BrowsingDataRemoverDelegate::EmbedderOriginTypeMatcher();
20 } 22 }
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
53 int remove_mask, 55 int remove_mask,
54 int origin_type_mask) { 56 int origin_type_mask) {
55 expected_calls_.emplace_back( 57 expected_calls_.emplace_back(
56 delete_begin, delete_end, remove_mask, origin_type_mask, 58 delete_begin, delete_end, remove_mask, origin_type_mask,
57 content::BrowsingDataFilterBuilder::Create( 59 content::BrowsingDataFilterBuilder::Create(
58 content::BrowsingDataFilterBuilder::BLACKLIST), 60 content::BrowsingDataFilterBuilder::BLACKLIST),
59 false /* should_compare_filter */); 61 false /* should_compare_filter */);
60 } 62 }
61 63
62 void MockBrowsingDataRemoverDelegate::VerifyAndClearExpectations() { 64 void MockBrowsingDataRemoverDelegate::VerifyAndClearExpectations() {
63 EXPECT_EQ(expected_calls_, actual_calls_); 65 EXPECT_EQ(expected_calls_.size(), actual_calls_.size())
66 << expected_calls_.size() << " calls were expected, but "
67 << actual_calls_.size() << " were made.";
68
69 if (expected_calls_.size() == actual_calls_.size()) {
70 auto actual = actual_calls_.begin();
71 int count = 0;
72 for (auto expected = expected_calls_.begin();
73 expected != expected_calls_.end(); expected++, actual++, count++) {
74 EXPECT_EQ(*expected, *actual) << "Call #" << count << " differs.";
75 }
76 }
77
64 expected_calls_.clear(); 78 expected_calls_.clear();
65 actual_calls_.clear(); 79 actual_calls_.clear();
66 } 80 }
67 81
68 MockBrowsingDataRemoverDelegate::CallParameters::CallParameters( 82 MockBrowsingDataRemoverDelegate::CallParameters::CallParameters(
69 const base::Time& delete_begin, 83 const base::Time& delete_begin,
70 const base::Time& delete_end, 84 const base::Time& delete_end,
71 int remove_mask, 85 int remove_mask,
72 int origin_type_mask, 86 int origin_type_mask,
73 std::unique_ptr<content::BrowsingDataFilterBuilder> filter_builder, 87 std::unique_ptr<content::BrowsingDataFilterBuilder> filter_builder,
(...skipping 15 matching lines...) Expand all
89 if (a.delete_begin_ != b.delete_begin_ || a.delete_end_ != b.delete_end_ || 103 if (a.delete_begin_ != b.delete_begin_ || a.delete_end_ != b.delete_end_ ||
90 a.remove_mask_ != b.remove_mask_ || 104 a.remove_mask_ != b.remove_mask_ ||
91 a.origin_type_mask_ != b.origin_type_mask_) { 105 a.origin_type_mask_ != b.origin_type_mask_) {
92 return false; 106 return false;
93 } 107 }
94 108
95 if (!a.should_compare_filter_ || !b.should_compare_filter_) 109 if (!a.should_compare_filter_ || !b.should_compare_filter_)
96 return true; 110 return true;
97 return *a.filter_builder_ == *b.filter_builder_; 111 return *a.filter_builder_ == *b.filter_builder_;
98 } 112 }
113
114 } // content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698