From 5280804b54f8b92d1f1d2f00730598aa974d3d42 Mon Sep 17 00:00:00 2001 From: Chris Thompson Date: Thu, 21 Jul 2022 00:08:23 +0000 Subject: [PATCH] Re-enable SCTAuditingHandlerTest.DisableReporting test After the fix for crbug.com/1344881 in crrev.com/c/3773419, this should no longer be prone to failure. Bug: 1341847 Change-Id: I5406a794499812973154611e416dd5af788b80dd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3778745 Auto-Submit: Chris Thompson Commit-Queue: Joe DeBlasio Reviewed-by: Joe DeBlasio Cr-Commit-Position: refs/heads/main@{#1026531} --- services/network/sct_auditing/sct_auditing_handler_unittest.cc | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/services/network/sct_auditing/sct_auditing_handler_unittest.cc b/services/network/sct_auditing/sct_auditing_handler_unittest.cc index 7176a3ea5e7b4b..a24c9ce3bef0db 100644 --- a/services/network/sct_auditing/sct_auditing_handler_unittest.cc +++ b/services/network/sct_auditing/sct_auditing_handler_unittest.cc @@ -221,8 +221,7 @@ void MakeTestSCTAndStatus( } // Tests that if reporting is disabled, reports are not created. -// crbug.com/1341847 Disable the test due to flakiness -TEST_F(SCTAuditingHandlerTest, DISABLED_DisableReporting) { +TEST_F(SCTAuditingHandlerTest, DisableReporting) { // Create a report which would normally trigger a send. const net::HostPortPair host_port_pair("example.com", 443); net::SignedCertificateTimestampAndStatusList sct_list;