[MD settings] disable clear browsing data while it is running

chrome/browser/browsing_data/browsing_data_remover.h

 // Copyright (c) 2012 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_H_
 #define CHROME_BROWSER_BROWSING_DATA_BROWSING_DATA_REMOVER_H_
 
 #include <stdint.h>
 
 #include <set>
@@ skipping 157 matching lines @@
     int origin_type_mask;
   };
 
   struct TimeRange {
     TimeRange(base::Time begin, base::Time end) : begin(begin), end(end) {}
 
     base::Time begin;
     base::Time end;
   };
 
-  // Observer is notified when the removal is done. Done means keywords have
-  // been deleted, cache cleared and all other tasks scheduled.
+  // Observer is notified when the removal is active and when it's done.
   class Observer {
    public:
-    virtual void OnBrowsingDataRemoverDone() = 0;
+    // Whether removal is active. Note that not having an active removal is not
+    // same as completing a removal. That is why the removing status is separate
+    // from the done message.
+    virtual void OnBrowsingDataRemoving(bool is_removing) {}

[msramek, 2016/07/11 17:00:01]

Please add a comment here to note that this method is DEPRECATED and not to be
used. I want to avoid that people start using it and I'll have to hunt down the
callsites.

[Dan Beam, 2016/07/11 21:10:57]

Done.

+
+    // Done means keywords have been deleted, cache cleared and all other
+    // removal tasks are scheduled.
+    virtual void OnBrowsingDataRemoverDone() {}
 
    protected:
     virtual ~Observer() {}
   };
 
   using Callback = base::Callback<void(const NotificationDetails&)>;
   using CallbackSubscription = std::unique_ptr<
       base::CallbackList<void(const NotificationDetails&)>::Subscription>;
 
   // The completion inhibitor can artificially delay completion of the browsing
@@ skipping 304 matching lines @@
   // not initialised, so the registry must be mocked out.
   std::unique_ptr<WebappRegistry> webapp_registry_;
 #endif
 
   base::WeakPtrFactory<BrowsingDataRemover> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowsingDataRemover);
 };
 
 #endif  // CHROME_BROWSER_BROWSING_DATA_BROWSING_DATA_REMOVER_H_