Add checkbox for reporting invalid TLS/SSL cert chains

chrome/browser/interstitials/security_interstitial_metrics_helper.h

 // Copyright (c) 2014 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_INTERSTITIALS_SECURITY_INTERSTITIAL_METRICS_HELPER_H_
 #define CHROME_BROWSER_INTERSTITIALS_SECURITY_INTERSTITIAL_METRICS_HELPER_H_
 
 #include <string>
 
 #include "base/task/cancelable_task_tracker.h"
@@ skipping 24 matching lines @@
     MAX_DECISION
   };
   enum SecurityInterstitialInteraction {
     TOTAL_VISITS,
     SHOW_ADVANCED,
     SHOW_PRIVACY_POLICY,
     SHOW_DIAGNOSTIC,
     SHOW_LEARN_MORE,
     RELOAD,
     OPEN_TIME_SETTINGS,
+    SET_EXTENDED_REPORTING_ENABLED_SSL,
+    SET_EXTENDED_REPORTING_ENABLED_SAFE_BROWSING,

[felt, 2015/03/25 01:07:17]

You don't need to distinguish between SSL and SAFE_BROWSING here. The metrics
helper records the same enum under different names for each type of warning.

[estark, 2015/03/25 04:08:02]

Done. Also, oops, I wasn't actually using SET_EXTENDED_REPORTING_ENABLED
anywhere; fixed that too. I also added SET_EXTENDED_REPORTING_DISABLED so that
we can see when users turn off extended reporting after opting in previously.
Does that sound about right to you?

+    EXTENDED_REPORTING_IS_ENABLED_SSL,
+    EXTENDED_REPORTING_IS_ENABLED_SAFE_BROWSING,
     MAX_INTERACTION
   };
 
   enum RapporReporting {
     REPORT_RAPPOR,
     SKIP_RAPPOR,
   };
 
   // Args:
   //   url: URL of page that triggered the interstitial. Only origin is used.
@@ skipping 30 matching lines @@
   int num_visits_;
   base::CancelableTaskTracker request_tracker_;
 #if defined(ENABLE_EXTENSIONS)
   scoped_ptr<extensions::ExperienceSamplingEvent> sampling_event_;
 #endif
 
   DISALLOW_COPY_AND_ASSIGN(SecurityInterstitialMetricsHelper);
 };
 
 #endif  // CHROME_BROWSER_INTERSTITIALS_SECURITY_INTERSTITIAL_METRICS_HELPER_H_