| Index: chrome/browser/ssl/captive_portal_blocking_page_browsertest.cc
|
| diff --git a/chrome/browser/ssl/captive_portal_blocking_page_browsertest.cc b/chrome/browser/ssl/captive_portal_blocking_page_browsertest.cc
|
| index 0d649ee633bc87fb176f524546a33baa85c1b2c0..6330b0e7d231f5491bc06e39c4263677351962ff 100644
|
| --- a/chrome/browser/ssl/captive_portal_blocking_page_browsertest.cc
|
| +++ b/chrome/browser/ssl/captive_portal_blocking_page_browsertest.cc
|
| @@ -279,14 +279,18 @@ IN_PROC_BROWSER_TEST_F(CaptivePortalBlockingPageTest,
|
| }
|
|
|
| IN_PROC_BROWSER_TEST_F(CaptivePortalBlockingPageTest, CertReportingOptIn) {
|
| - CertificateReportingTestUtils::SetCertReportingFinchConfig(
|
| - CertReportHelper::kFinchGroupShowPossiblySend, "1.0");
|
| - TestCertReporting(CertificateReportingTestUtils::EXTENDED_REPORTING_OPT_IN);
|
| + // This test should only run if the Finch config is such that reports
|
| + // will be sent when opted in. This tests that a report *is* sent when
|
| + // the user opts in under such a Finch config, and the below test
|
| + // tests that a report *is not* sent when the user doesn't opt in
|
| + // (under any Finch config).
|
| + if (CertificateReportingTestUtils::GetReportExpectedFromFinch() ==
|
| + CertificateReportingTestUtils::CERT_REPORT_EXPECTED) {
|
| + TestCertReporting(CertificateReportingTestUtils::EXTENDED_REPORTING_OPT_IN);
|
| + }
|
| }
|
|
|
| IN_PROC_BROWSER_TEST_F(CaptivePortalBlockingPageTest, CertReportingOptOut) {
|
| - CertificateReportingTestUtils::SetCertReportingFinchConfig(
|
| - CertReportHelper::kFinchGroupShowPossiblySend, "1.0");
|
| TestCertReporting(
|
| CertificateReportingTestUtils::EXTENDED_REPORTING_DO_NOT_OPT_IN);
|
| }
|
|
|