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

chrome/browser/browsing_data/browsing_data_remover.h

Index: chrome/browser/browsing_data/browsing_data_remover.h
diff --git a/chrome/browser/browsing_data/browsing_data_remover.h b/chrome/browser/browsing_data/browsing_data_remover.h
index 9e12be054b20b9017f3bf3c858329eb5e34f4cd3..ea6f4acf74f45e284ce57af8797975691944c913 100644
--- a/chrome/browser/browsing_data/browsing_data_remover.h
+++ b/chrome/browser/browsing_data/browsing_data_remover.h
@@ -175,11 +175,17 @@ class BrowsingDataRemover : public KeyedService
     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() {}