Remove old SBInterstitial counter

chrome/browser/safe_browsing/safe_browsing_blocking_page.cc

Index: chrome/browser/safe_browsing/safe_browsing_blocking_page.cc
diff --git a/chrome/browser/safe_browsing/safe_browsing_blocking_page.cc b/chrome/browser/safe_browsing/safe_browsing_blocking_page.cc
index 3bab81918b336bdb591ca1dde1c97f45350b95a2..6d111dd6283272d57761f3945d799fbae7ce519f 100644
--- a/chrome/browser/safe_browsing/safe_browsing_blocking_page.cc
+++ b/chrome/browser/safe_browsing/safe_browsing_blocking_page.cc
@@ -673,41 +673,6 @@ void SafeBrowsingBlockingPage::RecordUserAction(BlockingPageEvent event) {
       }
     }
   }
-
-  // TODO(felt): Get rid of the old interstitial histogram.
-  std::string action = "SBInterstitial";
-  switch (interstitial_type_) {
-    case TYPE_MALWARE_AND_PHISHING:
-      action.append("Multiple");
-      break;
-    case TYPE_MALWARE:
-      action.append("Malware");
-      break;
-    case TYPE_PHISHING:
-      action.append("Phishing");
-      break;
-  }
-
-  switch (event) {
-    case SHOW:
-      action.append("Show");
-      break;
-    case PROCEED:
-      action.append("Proceed");
-      break;
-    case DONT_PROCEED:
-      if (IsPrefEnabled(prefs::kSafeBrowsingProceedAnywayDisabled))
-        action.append("ForcedDontProceed");
-      else
-        action.append("DontProceed");
-      break;
-    case SHOW_ADVANCED:
-      break;
-    default:
-      NOTREACHED() << "Unexpected event: " << event;
-  }
-
-  content::RecordComputedAction(action);

[mattm, 2014/08/04 20:26:06]

This is not technically a histogram. (It's under the "counts" section of the uma
dashboard. The "SBInterstitial" also showing up in histogram_suffixes is
coincidental.) It's not only relevant to the v1 interstitial, it's just old.

Probably still want to get rid of it since the more useful actual histograms
SB2.InterstitialAction / SB2.InterstitialActionDetails also exist, but it
doesn't quite match the CL description.

 }
 
 void SafeBrowsingBlockingPage::RecordUserReactionTime(