Report FoundUwSReadError only once per Software Reporter run.

chrome/browser/safe_browsing/srt_fetcher_win.cc

 // Copyright 2015 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 "chrome/browser/safe_browsing/srt_fetcher_win.h"
 
 #include <stdint.h>
 
 #include <algorithm>
 #include <memory>
@@ skipping 254 matching lines @@
       if (base::StringToUint(uws_string, &uws_id)) {
         RecordSparseHistogram(kFoundUwsMetricName, uws_id);
         if (rappor_service) {
           rappor_service->RecordSampleString(kFoundUwsMetricName,
                                              rappor::COARSE_RAPPOR_TYPE,
                                              base::UTF16ToUTF8(uws_string));
         }
       } else {
         parse_error = true;
       }
+    }
 
-      // Clean up the old value.
-      reporter_key.DeleteValue(kFoundUwsValueName);
-
-      RecordBooleanHistogram(kFoundUwsReadErrorMetricName, parse_error);
-    }
+    // Clean up the old value.
+    reporter_key.DeleteValue(kFoundUwsValueName);
+    RecordBooleanHistogram(kFoundUwsReadErrorMetricName, parse_error);
   }
 
   // Reports to UMA the memory usage of the software reporter tool as reported
   // by the tool itself in the Windows registry.
   void ReportMemoryUsage() const {
     base::win::RegKey reporter_key;
     DWORD memory_used = 0;
     if (reporter_key.Open(HKEY_CURRENT_USER, registry_key_.c_str(),
                           KEY_QUERY_VALUE | KEY_SET_VALUE) != ERROR_SUCCESS ||
         reporter_key.ReadValueDW(kMemoryUsedValueName, &memory_used) !=
@@ skipping 808 matching lines @@
   return srt_cleaner_key.Open(HKEY_CURRENT_USER, cleaner_key_path.c_str(),
                               KEY_QUERY_VALUE) == ERROR_SUCCESS &&
          srt_cleaner_key.GetValueCount() > 0;
 }
 
 void SetSwReporterTestingDelegate(SwReporterTestingDelegate* delegate) {
   g_testing_delegate_ = delegate;
 }
 
 }  // namespace safe_browsing