Split BrowsingDataRemover into an abstract interface and implementation.

chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.h"
 
 #include <set>
 #include <string>
 #include <utility>
 
+#include "base/callback.h"
 #include "chrome/browser/autofill/personal_data_manager_factory.h"
 #include "chrome/browser/bookmarks/bookmark_model_factory.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/browsing_data/browsing_data_filter_builder.h"
 #include "chrome/browser/browsing_data/browsing_data_helper.h"
 #include "chrome/browser/browsing_data/registrable_domain_filter_builder.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/content_settings/host_content_settings_map_factory.h"
 #include "chrome/browser/domain_reliability/service_factory.h"
 #include "chrome/browser/download/download_prefs.h"
@@ skipping 180 matching lines @@
 void ClearHostnameResolutionCacheOnIOThread(
     IOThread* io_thread,
     base::Callback<bool(const std::string&)> host_filter) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
   io_thread->ClearHostCache(host_filter);
 }
 
 }  // namespace
 
+ChromeBrowsingDataRemoverDelegate::SubTask::SubTask(
+    const base::Closure& forward_callback)
+    : is_pending_(false),
+      forward_callback_(forward_callback),
+      weak_ptr_factory_(this) {
+  DCHECK(!forward_callback_.is_null());
+}
+
+ChromeBrowsingDataRemoverDelegate::SubTask::~SubTask() {}
+
+void ChromeBrowsingDataRemoverDelegate::SubTask::Start() {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  DCHECK(!is_pending_);
+  is_pending_ = true;
+}
+
+base::Closure
+ChromeBrowsingDataRemoverDelegate::SubTask::GetCompletionCallback() {
+  return base::Bind(&SubTask::CompletionCallback,
+                    weak_ptr_factory_.GetWeakPtr());
+}
+
+void ChromeBrowsingDataRemoverDelegate::SubTask::CompletionCallback() {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  DCHECK(is_pending_);
+  is_pending_ = false;
+  forward_callback_.Run();
+}
+
 ChromeBrowsingDataRemoverDelegate::ChromeBrowsingDataRemoverDelegate(
     BrowserContext* browser_context)
     : profile_(Profile::FromBrowserContext(browser_context)),
       sub_task_forward_callback_(
           base::Bind(&ChromeBrowsingDataRemoverDelegate::NotifyIfDone,
                      base::Unretained(this))),
       synchronous_clear_operations_(sub_task_forward_callback_),
       clear_autofill_origin_urls_(sub_task_forward_callback_),
       clear_flash_content_licenses_(sub_task_forward_callback_),
       clear_domain_reliability_monitor_(sub_task_forward_callback_),
@@ skipping 648 matching lines @@
 
 #if defined(OS_CHROMEOS)
 void ChromeBrowsingDataRemoverDelegate::OnClearPlatformKeys(
     chromeos::DBusMethodCallStatus call_status,
     bool result) {
   LOG_IF(ERROR, call_status != chromeos::DBUS_METHOD_CALL_SUCCESS || !result)
       << "Failed to clear platform keys.";
   clear_platform_keys_.GetCompletionCallback().Run();
 }
 #endif