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

Side by Side Diff: chrome/browser/browsing_data/mock_browsing_data_remover_delegate.h

Issue 2781083002: Fix the multi-threaded access to WeakPtr<BrowsingDataRemoverImpl> (Closed)
Patch Set: Static method to local namespace. Created 3 years, 8 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 #ifndef CHROME_BROWSER_BROWSING_DATA_MOCK_BROWSING_DATA_REMOVER_DELEGATE_H_ 5 #ifndef CHROME_BROWSER_BROWSING_DATA_MOCK_BROWSING_DATA_REMOVER_DELEGATE_H_
6 #define CHROME_BROWSER_BROWSING_DATA_MOCK_BROWSING_DATA_REMOVER_DELEGATE_H_ 6 #define CHROME_BROWSER_BROWSING_DATA_MOCK_BROWSING_DATA_REMOVER_DELEGATE_H_
7 7
8 #include <list> 8 #include <list>
9 #include <memory> 9 #include <memory>
10 10
11 #include "base/time/time.h" 11 #include "base/time/time.h"
12 #include "chrome/browser/browsing_data/browsing_data_remover_delegate.h" 12 #include "chrome/browser/browsing_data/browsing_data_remover_delegate.h"
13 #include "content/public/browser/browsing_data_filter_builder.h" 13 #include "content/public/browser/browsing_data_filter_builder.h"
14 14
15 // A BrowsingDataRemoverDelegate that only records RemoveEmbedderData() calls. 15 // A BrowsingDataRemoverDelegate that only records RemoveEmbedderData() calls.
16 class MockBrowsingDataRemoverDelegate : public BrowsingDataRemoverDelegate { 16 class MockBrowsingDataRemoverDelegate : public BrowsingDataRemoverDelegate {
17 public: 17 public:
18 MockBrowsingDataRemoverDelegate(); 18 MockBrowsingDataRemoverDelegate();
19 ~MockBrowsingDataRemoverDelegate() override; 19 ~MockBrowsingDataRemoverDelegate() override;
20 20
21 // BrowsingDataRemoverDelegate: 21 // BrowsingDataRemoverDelegate:
22 bool DoesOriginMatchEmbedderMask( 22 BrowsingDataRemoverDelegate::EmbedderOriginTypeMatcher GetOriginTypeMatcher()
23 int origin_type_mask, 23 const override;
24 const GURL& origin,
25 storage::SpecialStoragePolicy* special_storage_policy) const override;
26 void RemoveEmbedderData( 24 void RemoveEmbedderData(
27 const base::Time& delete_begin, 25 const base::Time& delete_begin,
28 const base::Time& delete_end, 26 const base::Time& delete_end,
29 int remove_mask, 27 int remove_mask,
30 const content::BrowsingDataFilterBuilder& filter_builder, 28 const content::BrowsingDataFilterBuilder& filter_builder,
31 int origin_type_mask, 29 int origin_type_mask,
32 const base::Closure& callback) override; 30 const base::Closure& callback) override;
33 31
34 // Add an expected call for testing. 32 // Add an expected call for testing.
35 void ExpectCall(const base::Time& delete_begin, 33 void ExpectCall(const base::Time& delete_begin,
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
68 int origin_type_mask_; 66 int origin_type_mask_;
69 std::unique_ptr<content::BrowsingDataFilterBuilder> filter_builder_; 67 std::unique_ptr<content::BrowsingDataFilterBuilder> filter_builder_;
70 bool should_compare_filter_; 68 bool should_compare_filter_;
71 }; 69 };
72 70
73 std::list<CallParameters> actual_calls_; 71 std::list<CallParameters> actual_calls_;
74 std::list<CallParameters> expected_calls_; 72 std::list<CallParameters> expected_calls_;
75 }; 73 };
76 74
77 #endif // CHROME_BROWSER_BROWSING_DATA_MOCK_BROWSING_DATA_REMOVER_DELEGATE_H_ 75 #endif // CHROME_BROWSER_BROWSING_DATA_MOCK_BROWSING_DATA_REMOVER_DELEGATE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698