Add Finch testing configs for ReportCertificateErrors study

chrome/browser/ssl/ssl_browser_tests.cc

 // Copyright (c) 2012 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.
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "base/location.h"
 #include "base/metrics/field_trial.h"
@@ skipping 1047 matching lines @@
       &replacement_path));
 
   // Load a page that displays insecure content.
   ui_test_utils::NavigateToURL(browser(),
                                https_server_.GetURL(replacement_path));
 
   CheckAuthenticatedState(browser()->tab_strip_model()->GetActiveWebContents(),
                           AuthState::DISPLAYED_INSECURE_CONTENT);
 }
 
-// User proceeds, checkbox is shown and checked, Finch parameter is set
-// -> we expect a report.
-IN_PROC_BROWSER_TEST_F(
-    SSLUITestWithExtendedReporting,
-    TestBrokenHTTPSProceedWithShowYesCheckYesParamYesReportYes) {
-  CertificateReportingTestUtils::SetCertReportingFinchConfig(
-      CertReportHelper::kFinchGroupShowPossiblySend, "1.0");
+// Test that if the user proceeds and the checkbox is checked, a report
+// is sent or not sent depending on the Finch config.
+IN_PROC_BROWSER_TEST_F(SSLUITestWithExtendedReporting,
+                       TestBrokenHTTPSProceedReporting) {
+  CertificateReportingTestUtils::ExpectReport expect_report =
+      CertificateReportingTestUtils::GetReportExpectedFromFinch();
   TestBrokenHTTPSReporting(
       CertificateReportingTestUtils::EXTENDED_REPORTING_OPT_IN,
-      CertificateReportingTestUtils::SSL_INTERSTITIAL_PROCEED,
-      CertificateReportingTestUtils::CERT_REPORT_EXPECTED, browser());
+      CertificateReportingTestUtils::SSL_INTERSTITIAL_PROCEED, expect_report,
+      browser());
 }
 
-// User goes back, checkbox is shown and checked, Finch parameter is set
-// -> we expect a report.
-IN_PROC_BROWSER_TEST_F(
-    SSLUITestWithExtendedReporting,
-    TestBrokenHTTPSGoBackWithShowYesCheckYesParamYesReportYes) {
-  CertificateReportingTestUtils::SetCertReportingFinchConfig(
-      CertReportHelper::kFinchGroupShowPossiblySend, "1.0");
+// Test that if the user goes back and the checkbox is checked, a report
+// is sent or not sent depending on the Finch config.
+IN_PROC_BROWSER_TEST_F(SSLUITestWithExtendedReporting,
+                       TestBrokenHTTPSGoBackReporting) {
+  CertificateReportingTestUtils::ExpectReport expect_report =
+      CertificateReportingTestUtils::GetReportExpectedFromFinch();
   TestBrokenHTTPSReporting(
       CertificateReportingTestUtils::EXTENDED_REPORTING_OPT_IN,
       CertificateReportingTestUtils::SSL_INTERSTITIAL_DO_NOT_PROCEED,
-      CertificateReportingTestUtils::CERT_REPORT_EXPECTED, browser());
+      expect_report, browser());
 }
 
-// User proceeds, checkbox is shown but unchecked, Finch parameter is set
-// -> we expect no report.
-IN_PROC_BROWSER_TEST_F(
-    SSLUITestWithExtendedReporting,
-    TestBrokenHTTPSProceedWithShowYesCheckNoParamYesReportNo) {
-  CertificateReportingTestUtils::SetCertReportingFinchConfig(
-      CertReportHelper::kFinchGroupShowPossiblySend, "1.0");
+// User proceeds, checkbox is shown but unchecked. Reports should never
+// be sent, regardless of Finch config.
+IN_PROC_BROWSER_TEST_F(SSLUITestWithExtendedReporting,
+                       TestBrokenHTTPSProceedReportingWithNoOptIn) {
   TestBrokenHTTPSReporting(
       CertificateReportingTestUtils::EXTENDED_REPORTING_DO_NOT_OPT_IN,
       CertificateReportingTestUtils::SSL_INTERSTITIAL_PROCEED,
       CertificateReportingTestUtils::CERT_REPORT_NOT_EXPECTED, browser());
 }
 
-// User goes back, checkbox is shown but unchecked, Finch parameter is set
-// -> we expect no report.
+// User goes back, checkbox is shown but unchecked. Reports should never
+// be sent, regardless of Finch config.
 IN_PROC_BROWSER_TEST_F(SSLUITestWithExtendedReporting,
                        TestBrokenHTTPSGoBackShowYesCheckNoParamYesReportNo) {
-  CertificateReportingTestUtils::SetCertReportingFinchConfig(
-      CertReportHelper::kFinchGroupShowPossiblySend, "1.0");
   TestBrokenHTTPSReporting(
       CertificateReportingTestUtils::EXTENDED_REPORTING_DO_NOT_OPT_IN,
       CertificateReportingTestUtils::SSL_INTERSTITIAL_DO_NOT_PROCEED,
       CertificateReportingTestUtils::CERT_REPORT_NOT_EXPECTED, browser());
 }
 
-// User proceeds, checkbox is shown and checked, Finch parameter is not
-// set -> we expect no report.
+// User proceeds, checkbox is not shown but checked -> we expect no
+// report.
 IN_PROC_BROWSER_TEST_F(SSLUITestWithExtendedReporting,
-                       TestBrokenHTTPSProceedShowYesCheckYesParamNoReportNo) {
-  CertificateReportingTestUtils::SetCertReportingFinchConfig(
-      CertReportHelper::kFinchGroupShowPossiblySend, "-1.0");
+                       TestBrokenHTTPSProceedShowNoCheckYesReportNo) {
+  if (base::FieldTrialList::FindFullName(
+          CertReportHelper::kFinchExperimentName) ==
+      CertReportHelper::kFinchGroupDontShowDontSend) {
+    TestBrokenHTTPSReporting(
+        CertificateReportingTestUtils::EXTENDED_REPORTING_OPT_IN,
+        CertificateReportingTestUtils::SSL_INTERSTITIAL_PROCEED,
+        CertificateReportingTestUtils::CERT_REPORT_NOT_EXPECTED, browser());
+  }
+}
+
+// Browser is incognito, user proceeds, checkbox has previously opted in
+// -> no report, regardless of Finch config.
+IN_PROC_BROWSER_TEST_F(SSLUITestWithExtendedReporting,
+                       TestBrokenHTTPSInIncognitoReportNo) {
   TestBrokenHTTPSReporting(
       CertificateReportingTestUtils::EXTENDED_REPORTING_OPT_IN,
       CertificateReportingTestUtils::SSL_INTERSTITIAL_PROCEED,
-      CertificateReportingTestUtils::CERT_REPORT_NOT_EXPECTED, browser());
-}
-
-// User goes back, checkbox is shown and checked, Finch parameter is not set
-// -> we expect no report.
-IN_PROC_BROWSER_TEST_F(SSLUITestWithExtendedReporting,
-                       TestBrokenHTTPSGoBackShowYesCheckYesParamNoReportNo) {
-  CertificateReportingTestUtils::SetCertReportingFinchConfig(
-      CertReportHelper::kFinchGroupShowPossiblySend, "-1.0");
-  TestBrokenHTTPSReporting(
-      CertificateReportingTestUtils::EXTENDED_REPORTING_OPT_IN,
-      CertificateReportingTestUtils::SSL_INTERSTITIAL_DO_NOT_PROCEED,
-      CertificateReportingTestUtils::CERT_REPORT_NOT_EXPECTED, browser());
-}
-
-// User proceeds, checkbox is not shown but checked -> we expect no report
-IN_PROC_BROWSER_TEST_F(SSLUITestWithExtendedReporting,
-                       TestBrokenHTTPSProceedShowNoCheckYesReportNo) {
-  CertificateReportingTestUtils::SetCertReportingFinchConfig(
-      CertReportHelper::kFinchGroupDontShowDontSend);
-  TestBrokenHTTPSReporting(
-      CertificateReportingTestUtils::EXTENDED_REPORTING_OPT_IN,
-      CertificateReportingTestUtils::SSL_INTERSTITIAL_PROCEED,
-      CertificateReportingTestUtils::CERT_REPORT_NOT_EXPECTED, browser());
-}
-
-// Browser is incognito, user proceeds, checkbox is shown and checked, Finch
-// parameter is set -> we expect no report
-IN_PROC_BROWSER_TEST_F(SSLUITestWithExtendedReporting,
-                       TestBrokenHTTPSInIncognitoReportNo) {
-  CertificateReportingTestUtils::SetCertReportingFinchConfig(
-      CertReportHelper::kFinchGroupShowPossiblySend, "1.0");
-  TestBrokenHTTPSReporting(
-      CertificateReportingTestUtils::EXTENDED_REPORTING_OPT_IN,
-      CertificateReportingTestUtils::SSL_INTERSTITIAL_PROCEED,
       CertificateReportingTestUtils::CERT_REPORT_NOT_EXPECTED,
       CreateIncognitoBrowser());
 }
 
-// User proceeds, checkbox is shown and checked, Finch parameter is invalid
-// -> we expect no report.
-IN_PROC_BROWSER_TEST_F(
-    SSLUITestWithExtendedReporting,
-    TestBrokenHTTPSProceedWithShowYesCheckYesParamInvalidReportNo) {
-  CertificateReportingTestUtils::SetCertReportingFinchConfig(
-      CertReportHelper::kFinchGroupShowPossiblySend, "abcdef");
-  TestBrokenHTTPSReporting(
-      CertificateReportingTestUtils::EXTENDED_REPORTING_OPT_IN,
-      CertificateReportingTestUtils::SSL_INTERSTITIAL_PROCEED,
-      CertificateReportingTestUtils::CERT_REPORT_NOT_EXPECTED, browser());
-}
-
 // Test that reports don't get sent when extended reporting opt-in is
 // disabled by policy.
 IN_PROC_BROWSER_TEST_F(SSLUITestWithExtendedReporting,
                        TestBrokenHTTPSNoReportingWhenDisallowed) {
   browser()->profile()->GetPrefs()->SetBoolean(
       prefs::kSafeBrowsingExtendedReportingOptInAllowed, false);
   TestBrokenHTTPSReporting(
       CertificateReportingTestUtils::EXTENDED_REPORTING_OPT_IN,
       CertificateReportingTestUtils::SSL_INTERSTITIAL_PROCEED,
       CertificateReportingTestUtils::CERT_REPORT_NOT_EXPECTED, browser());
@@ skipping 1057 matching lines @@
 
 // Visit a page over https that contains a frame with a redirect.
 
 // XMLHttpRequest insecure content in synchronous mode.
 
 // XMLHttpRequest insecure content in asynchronous mode.
 
 // XMLHttpRequest over bad ssl in synchronous mode.
 
 // XMLHttpRequest over OK ssl in synchronous mode.