Scrub the vestigal breakpad crash dump attempts metrics code.

chrome/app/chrome_crash_reporter_client_win.h

Index: chrome/app/chrome_crash_reporter_client_win.h
diff --git a/chrome/app/chrome_crash_reporter_client_win.h b/chrome/app/chrome_crash_reporter_client_win.h
index 7fcec6df364b837d99126b47d73972dc8c20358b..bcf7f7974ce455448fdca737c386c905b648d293 100644
--- a/chrome/app/chrome_crash_reporter_client_win.h
+++ b/chrome/app/chrome_crash_reporter_client_win.h
@@ -13,10 +13,6 @@
 #include "build/build_config.h"

[grt (UTC plus 2), 2016/04/26 17:01:33]

unused

[Sigurður Ásgeirsson, 2016/04/26 17:28:03]

Done.

 #include "components/crash/content/app/crash_reporter_client.h"
 
-namespace browser_watcher {
-class CrashReportingMetrics;
-}  // namespace browser_watcher
-
 class ChromeCrashReporterClient : public crash_reporter::CrashReporterClient {
  public:
   ChromeCrashReporterClient();
@@ -37,7 +33,6 @@ class ChromeCrashReporterClient : public crash_reporter::CrashReporterClient {
   bool GetIsPerUserInstall(const base::FilePath& exe_path) override;
   bool GetShouldDumpLargerDumps(bool is_per_user_install) override;
   int GetResultCodeRespawnFailed() override;
-  void InitBrowserCrashDumpsRegKey() override;
   void RecordCrashDumpAttempt(bool is_real_crash) override;

[grt (UTC plus 2), 2016/04/26 17:01:33]

could these be removed from CrashReporterClient altogether?

[Sigurður Ásgeirsson, 2016/04/26 17:28:03]

Yups - so removed.

   void RecordCrashDumpAttemptResult(bool is_real_crash,
                                     bool succeeded) override;
@@ -55,9 +50,6 @@ class ChromeCrashReporterClient : public crash_reporter::CrashReporterClient {
   bool EnableBreakpadForProcess(const std::string& process_type) override;
 
  private:
-  std::unique_ptr<browser_watcher::CrashReportingMetrics>
-      crash_reporting_metrics_;
-
   DISALLOW_COPY_AND_ASSIGN(ChromeCrashReporterClient);
 };