Revert of Send a TrackedPreference incident when registry pref validation fails.

components/user_prefs/tracked/tracked_atomic_preference.cc

 // Copyright 2014 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 "components/user_prefs/tracked/tracked_atomic_preference.h"
 
 #include "base/values.h"
 #include "components/user_prefs/tracked/pref_hash_store_transaction.h"
 #include "components/user_prefs/tracked/tracked_preference_validation_delegate.h"
 
@@ skipping 27 matching lines @@
     base::DictionaryValue* pref_store_contents,
     PrefHashStoreTransaction* transaction,
     PrefHashStoreTransaction* external_validation_transaction) const {
   const base::Value* value = NULL;
   pref_store_contents->Get(pref_path_, &value);
   PrefHashStoreTransaction::ValueState value_state =
       transaction->CheckValue(pref_path_, value);
   helper_.ReportValidationResult(value_state, transaction->GetStoreUMASuffix());
 
   PrefHashStoreTransaction::ValueState external_validation_value_state =
-      PrefHashStoreTransaction::UNSUPPORTED;
+      PrefHashStoreTransaction::UNCHANGED;
   if (external_validation_transaction) {
     external_validation_value_state =
         external_validation_transaction->CheckValue(pref_path_, value);
     helper_.ReportValidationResult(
         external_validation_value_state,
         external_validation_transaction->GetStoreUMASuffix());
+
+    // TODO(proberge): Call delegate_->OnAtomicPreferenceValidation.
   }
 
   if (delegate_) {
     delegate_->OnAtomicPreferenceValidation(pref_path_, value, value_state,
-                                            external_validation_value_state,
                                             helper_.IsPersonal());
   }
   TrackedPreferenceHelper::ResetAction reset_action =
       helper_.GetAction(value_state);
   helper_.ReportAction(reset_action);
 
   bool was_reset = false;
   if (reset_action == TrackedPreferenceHelper::DO_RESET) {
     pref_store_contents->RemovePath(pref_path_, NULL);
     was_reset = true;
@@ skipping 12 matching lines @@
       (was_reset ||
        external_validation_value_state !=
            PrefHashStoreTransaction::UNCHANGED)) {
     const base::Value* new_value = nullptr;
     pref_store_contents->Get(pref_path_, &new_value);
     external_validation_transaction->StoreHash(pref_path_, new_value);
   }
 
   return was_reset;
 }