Move BrowsingDataRemoverImpl:: CompletionInhibitor to the public interface

chrome/browser/browsing_data/browsing_data_remover_test_util.h

 // Copyright 2014 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.
 
 #ifndef CHROME_BROWSER_BROWSING_DATA_BROWSING_DATA_REMOVER_TEST_UTIL_H_
 #define CHROME_BROWSER_BROWSING_DATA_BROWSING_DATA_REMOVER_TEST_UTIL_H_
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/scoped_observer.h"
 #include "chrome/browser/browsing_data/browsing_data_remover.h"
-#include "chrome/browser/browsing_data/browsing_data_remover_impl.h"
 #include "content/public/test/test_utils.h"
 
 // This class can be used to wait for a BrowsingDataRemover to complete
 // operation. It is not suitable for repeated use.
 class BrowsingDataRemoverCompletionObserver
     : public BrowsingDataRemover::Observer {
  public:
   explicit BrowsingDataRemoverCompletionObserver(BrowsingDataRemover* remover);
   ~BrowsingDataRemoverCompletionObserver() override;
 
   void BlockUntilCompletion();
 
  protected:
   // BrowsingDataRemover::Observer:
   void OnBrowsingDataRemoverDone() override;
 
  private:
   scoped_refptr<content::MessageLoopRunner> message_loop_runner_;
   ScopedObserver<BrowsingDataRemover, BrowsingDataRemover::Observer> observer_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowsingDataRemoverCompletionObserver);
 };
 
-class BrowsingDataRemoverCompletionInhibitor
-    : public BrowsingDataRemoverImpl::CompletionInhibitor {
+// The completion inhibitor can artificially delay completion of the browsing
+// data removal process. It is used during testing to simulate scenarios in
+// which the deletion stalls or takes a very long time.
+//
+// This class will detach itself from |remover| upon its destruction.
+// If |remover| is destroyed during a test (e.g. in profile shutdown tests),
+// users must call Reset() to detach it in advance.
+class BrowsingDataRemoverCompletionInhibitor {
  public:
-  BrowsingDataRemoverCompletionInhibitor();
-  ~BrowsingDataRemoverCompletionInhibitor() override;
+  explicit BrowsingDataRemoverCompletionInhibitor(BrowsingDataRemover* remover);
+  virtual ~BrowsingDataRemoverCompletionInhibitor();
+
+  void Reset();
 
   void BlockUntilNearCompletion();
   void ContinueToCompletion();
 
  protected:
-  // BrowsingDataRemoverImpl::CompletionInhibitor:
-  void OnBrowsingDataRemoverWouldComplete(
-      BrowsingDataRemoverImpl* remover,
-      const base::Closure& continue_to_completion) override;
+  virtual void OnBrowsingDataRemoverWouldComplete(
+      const base::Closure& continue_to_completion);
 
  private:
+  // Not owned by this class. If the pointer becomes invalid, the owner of
+  // this class is responsible for calling Reset().
+  BrowsingDataRemover* remover_;
+
   scoped_refptr<content::MessageLoopRunner> message_loop_runner_;
   base::Closure continue_to_completion_callback_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowsingDataRemoverCompletionInhibitor);
 };
 
 #endif  // CHROME_BROWSER_BROWSING_DATA_BROWSING_DATA_REMOVER_TEST_UTIL_H_