Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2933)

Unified Diff: chrome/browser/safe_browsing/incident_reporting/incident_reporting_service_unittest.cc

Issue 2692843002: Fail tests fast if overriding the Windows registry fails. (Closed)
Patch Set: sync to position 450085 Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/safe_browsing/incident_reporting/incident_reporting_service_unittest.cc
diff --git a/chrome/browser/safe_browsing/incident_reporting/incident_reporting_service_unittest.cc b/chrome/browser/safe_browsing/incident_reporting/incident_reporting_service_unittest.cc
index 8fd8e3b23bb164023bf152407fd4990436246a2a..4a3980f54c094e9ae73c52ba4ad90af5d327c2c3 100644
--- a/chrome/browser/safe_browsing/incident_reporting/incident_reporting_service_unittest.cc
+++ b/chrome/browser/safe_browsing/incident_reporting/incident_reporting_service_unittest.cc
@@ -212,7 +212,8 @@ class IncidentReportingServiceTest : public testing::Test {
#if defined(OS_WIN)
// Redirect HKCU so that the platform state store used by the test doesn't
// collide with existing Chrome installs or other tests running in parallel.
- registry_override_manager_.OverrideRegistry(HKEY_CURRENT_USER);
+ ASSERT_NO_FATAL_FAILURE(
+ registry_override_manager_.OverrideRegistry(HKEY_CURRENT_USER));
#endif
ASSERT_TRUE(profile_manager_.SetUp());
}

Powered by Google App Engine
This is Rietveld 408576698