Log basic/advanced tab action from java

components/browsing_data/core/browsing_data_utils.cc

Index: components/browsing_data/core/browsing_data_utils.cc
diff --git a/components/browsing_data/core/browsing_data_utils.cc b/components/browsing_data/core/browsing_data_utils.cc
index 7dba88ed150cf7ccaeb3ceefd2acac7014db1d0e..5e04932b640cd8d2e9e1534e0ef3aabefba06c76 100644
--- a/components/browsing_data/core/browsing_data_utils.cc
+++ b/components/browsing_data/core/browsing_data_utils.cc
@@ -91,22 +91,6 @@ void RecordTimePeriodChange(TimePeriod period) {
   }
 }
 
-void RecordDeletionFromTab(ClearBrowsingDataTab tab) {
-  UMA_HISTOGRAM_ENUMERATION("History.ClearBrowsingData.UserDeletedFromTab", tab,

[Bernhard Bauer, 2017/06/29 17:13:18]

Is this histogram now unused? In that case you should probably mark it as
obsolete.

[dullweber, 2017/06/30 08:48:35]

Thanks for noticing this. I forgot to add the histogram again.

-                            browsing_data::ClearBrowsingDataTab::NUM_TYPES);
-  switch (tab) {
-    case ClearBrowsingDataTab::BASIC:
-      base::RecordAction(base::UserMetricsAction("ClearBrowsingData_BasicTab"));
-      break;
-    case ClearBrowsingDataTab::ADVANCED:
-      base::RecordAction(
-          base::UserMetricsAction("ClearBrowsingData_AdvancedTab"));
-      break;
-    case ClearBrowsingDataTab::NUM_TYPES:
-      NOTREACHED();
-  }
-}
-
 base::string16 GetCounterTextFromResult(
     const BrowsingDataCounter::Result* result) {
   base::string16 text;