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

Unified Diff: chrome/browser/browsing_data/browsing_data_remover_test_util.cc

Issue 2802013002: Move BrowsingDataRemoverImpl:: CompletionInhibitor to the public interface (Closed)
Patch Set: Finishing after shutdown is still OK. 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/browsing_data/browsing_data_remover_test_util.cc
diff --git a/chrome/browser/browsing_data/browsing_data_remover_test_util.cc b/chrome/browser/browsing_data/browsing_data_remover_test_util.cc
index 9bdc5a399774bed2d039ade83657402c2fa234a9..6677ff7da225991bb6c737e6561cd60f019580cd 100644
--- a/chrome/browser/browsing_data/browsing_data_remover_test_util.cc
+++ b/chrome/browser/browsing_data/browsing_data_remover_test_util.cc
@@ -22,14 +22,27 @@ void BrowsingDataRemoverCompletionObserver::OnBrowsingDataRemoverDone() {
message_loop_runner_->Quit();
}
-BrowsingDataRemoverCompletionInhibitor::BrowsingDataRemoverCompletionInhibitor()
- : message_loop_runner_(new content::MessageLoopRunner) {
- BrowsingDataRemoverImpl::set_completion_inhibitor_for_testing(this);
+BrowsingDataRemoverCompletionInhibitor::BrowsingDataRemoverCompletionInhibitor(
+ BrowsingDataRemover* remover)
+ : remover_(remover), message_loop_runner_(new content::MessageLoopRunner) {
+ DCHECK(remover);
+ remover_->SetWouldCompleteCallbackForTesting(
+ base::Bind(&BrowsingDataRemoverCompletionInhibitor::
+ OnBrowsingDataRemoverWouldComplete,
+ base::Unretained(this)));
}
BrowsingDataRemoverCompletionInhibitor::
~BrowsingDataRemoverCompletionInhibitor() {
- BrowsingDataRemoverImpl::set_completion_inhibitor_for_testing(nullptr);
+ Reset();
+}
+
+void BrowsingDataRemoverCompletionInhibitor::Reset() {
+ if (!remover_)
+ return;
+ remover_->SetWouldCompleteCallbackForTesting(
+ base::Callback<void(const base::Closure&)>());
+ remover_ = nullptr;
}
void BrowsingDataRemoverCompletionInhibitor::BlockUntilNearCompletion() {
@@ -44,7 +57,6 @@ void BrowsingDataRemoverCompletionInhibitor::ContinueToCompletion() {
}
void BrowsingDataRemoverCompletionInhibitor::OnBrowsingDataRemoverWouldComplete(
- BrowsingDataRemoverImpl* remover,
const base::Closure& continue_to_completion) {
DCHECK(continue_to_completion_callback_.is_null());
continue_to_completion_callback_ = continue_to_completion;
« no previous file with comments | « chrome/browser/browsing_data/browsing_data_remover_test_util.h ('k') | chrome/browser/profiles/profile_manager_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698