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

chrome/browser/safe_browsing/incident_reporting/preference_validation_delegate_unittest.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 "chrome/browser/safe_browsing/incident_reporting/preference_validation_delegate.h"
 
 #include <stddef.h>
 
 #include <memory>
 #include <string>
@@ skipping 32 matching lines @@
     std::unique_ptr<safe_browsing::MockIncidentReceiver> receiver(
         new NiceMock<safe_browsing::MockIncidentReceiver>());
     ON_CALL(*receiver, DoAddIncidentForProfile(IsNull(), _))
         .WillByDefault(WithArg<1>(TakeIncidentToVector(&incidents_)));
     instance_.reset(new safe_browsing::PreferenceValidationDelegate(
         nullptr, std::move(receiver)));
   }
 
   static void ExpectValueStatesEquate(
       PrefHashStoreTransaction::ValueState store_state,
-      PrefHashStoreTransaction::ValueState external_validation_value_state,
       safe_browsing::
           ClientIncidentReport_IncidentData_TrackedPreferenceIncident_ValueState
               incident_state) {
     typedef safe_browsing::
         ClientIncidentReport_IncidentData_TrackedPreferenceIncident TPIncident;
     switch (store_state) {
       case PrefHashStoreTransaction::CLEARED:
         EXPECT_EQ(TPIncident::CLEARED, incident_state);
         break;
       case PrefHashStoreTransaction::CHANGED:
         EXPECT_EQ(TPIncident::CHANGED, incident_state);
         break;
       case PrefHashStoreTransaction::UNTRUSTED_UNKNOWN_VALUE:
         EXPECT_EQ(TPIncident::UNTRUSTED_UNKNOWN_VALUE, incident_state);
         break;
       default:
-        if (external_validation_value_state ==
-            PrefHashStoreTransaction::CLEARED) {
-          EXPECT_EQ(TPIncident::BYPASS_CLEARED, incident_state);
-        } else if (external_validation_value_state ==
-                   PrefHashStoreTransaction::CHANGED) {
-          EXPECT_EQ(TPIncident::BYPASS_CHANGED, incident_state);
-        } else {
-          FAIL() << "unexpected store state";
-        }
+        FAIL() << "unexpected store state";
         break;
     }
   }
 
   static void ExpectKeysEquate(
       const std::vector<std::string>& store_keys,
       const google::protobuf::RepeatedPtrField<std::string>& incident_keys) {
     ASSERT_EQ(store_keys.size(), static_cast<size_t>(incident_keys.size()));
     for (int i = 0; i < incident_keys.size(); ++i) {
       EXPECT_EQ(store_keys[i], incident_keys.Get(i));
     }
   }
 
   const std::string kPrefPath_;
   IncidentVector incidents_;
   std::unique_ptr<base::Value> null_value_;
   base::DictionaryValue dict_value_;
   std::vector<std::string> invalid_keys_;
   std::unique_ptr<TrackedPreferenceValidationDelegate> instance_;
 };
 
 // Tests that a NULL value results in an incident with no value.
 TEST_F(PreferenceValidationDelegateTest, NullValue) {
-  instance_->OnAtomicPreferenceValidation(
-      kPrefPath_, NULL, PrefHashStoreTransaction::CLEARED,
-      PrefHashStoreTransaction::UNSUPPORTED, false /* is_personal */);
+  instance_->OnAtomicPreferenceValidation(kPrefPath_,
+                                          NULL,
+                                          PrefHashStoreTransaction::CLEARED,
+                                          false /* is_personal */);
   std::unique_ptr<safe_browsing::ClientIncidentReport_IncidentData> incident(
       incidents_.back()->TakePayload());
   EXPECT_FALSE(incident->tracked_preference().has_atomic_value());
   EXPECT_EQ(
       safe_browsing::
           ClientIncidentReport_IncidentData_TrackedPreferenceIncident::CLEARED,
       incident->tracked_preference().value_state());
 }
 
 // Tests that all supported value types can be stringified into an incident. The
@@ skipping 40 matching lines @@
         ADD_FAILURE() << "unsupported value type " << value_type;
     }
     return std::unique_ptr<Value>();
   }
 
   base::Value::Type value_type_;
   const char* expected_value_;
 };
 
 TEST_P(PreferenceValidationDelegateValues, Value) {
-  instance_->OnAtomicPreferenceValidation(
-      kPrefPath_, MakeValue(value_type_).get(),
-      PrefHashStoreTransaction::CLEARED, PrefHashStoreTransaction::UNSUPPORTED,
-      false /* is_personal */);
+  instance_->OnAtomicPreferenceValidation(kPrefPath_,
+                                          MakeValue(value_type_).get(),
+                                          PrefHashStoreTransaction::CLEARED,
+                                          false /* is_personal */);
   ASSERT_EQ(1U, incidents_.size());
   std::unique_ptr<safe_browsing::ClientIncidentReport_IncidentData> incident(
       incidents_.back()->TakePayload());
   EXPECT_EQ(std::string(expected_value_),
             incident->tracked_preference().atomic_value());
 }
 
 INSTANTIATE_TEST_CASE_P(
     Values,
     PreferenceValidationDelegateValues,
@@ skipping 12 matching lines @@
         std::tr1::make_tuple(base::Value::TYPE_STRING,
                              const_cast<char*>("i have a spleen")),
         std::tr1::make_tuple(base::Value::TYPE_DICTIONARY,
             const_cast<char*>("{\"forty-seven\":47,\"twenty-two\":22}")),
         std::tr1::make_tuple(base::Value::TYPE_LIST,
                              const_cast<char*>("[22,47]"))));
 
 // Tests that no incidents are reported for relevant combinations of ValueState.
 class PreferenceValidationDelegateNoIncident
     : public PreferenceValidationDelegateTest,
-      public testing::WithParamInterface<
-          std::tr1::tuple<PrefHashStoreTransaction::ValueState,
-                          PrefHashStoreTransaction::ValueState>> {
+      public testing::WithParamInterface<PrefHashStoreTransaction::ValueState> {
  protected:
   void SetUp() override {
     PreferenceValidationDelegateTest::SetUp();
-    value_state_ = std::tr1::get<0>(GetParam());
-    external_validation_value_state_ = std::tr1::get<1>(GetParam());
+    value_state_ = GetParam();
   }
 
   PrefHashStoreTransaction::ValueState value_state_;
-  PrefHashStoreTransaction::ValueState external_validation_value_state_;
 };
 
 TEST_P(PreferenceValidationDelegateNoIncident, Atomic) {
-  instance_->OnAtomicPreferenceValidation(
-      kPrefPath_, null_value_.get(), value_state_,
-      external_validation_value_state_, false /* is_personal */);
+  instance_->OnAtomicPreferenceValidation(kPrefPath_,
+                                          null_value_.get(),
+                                          value_state_,
+                                          false /* is_personal */);
   EXPECT_EQ(0U, incidents_.size());
 }
 
 TEST_P(PreferenceValidationDelegateNoIncident, Split) {
-  instance_->OnSplitPreferenceValidation(
-      kPrefPath_, &dict_value_, invalid_keys_, value_state_,
-      external_validation_value_state_, false /* is_personal */);
+  instance_->OnSplitPreferenceValidation(kPrefPath_,
+                                         &dict_value_,
+                                         invalid_keys_,
+                                         value_state_,
+                                         false /* is_personal */);
   EXPECT_EQ(0U, incidents_.size());
 }
 
 INSTANTIATE_TEST_CASE_P(
     NoIncident,
     PreferenceValidationDelegateNoIncident,
-    testing::Combine(
-        testing::Values(PrefHashStoreTransaction::UNCHANGED,
-                        PrefHashStoreTransaction::SECURE_LEGACY,
-                        PrefHashStoreTransaction::TRUSTED_UNKNOWN_VALUE),
-        testing::Values(PrefHashStoreTransaction::UNCHANGED,
-                        PrefHashStoreTransaction::UNSUPPORTED,
-                        PrefHashStoreTransaction::UNTRUSTED_UNKNOWN_VALUE)));
+    testing::Values(PrefHashStoreTransaction::UNCHANGED,
+                    PrefHashStoreTransaction::SECURE_LEGACY,
+                    PrefHashStoreTransaction::TRUSTED_UNKNOWN_VALUE));
 
 // Tests that incidents are reported for relevant combinations of ValueState and
 // impersonal/personal.
 class PreferenceValidationDelegateWithIncident
     : public PreferenceValidationDelegateTest,
       public testing::WithParamInterface<
-          std::tr1::tuple<PrefHashStoreTransaction::ValueState,
-                          PrefHashStoreTransaction::ValueState,
-                          bool>> {
+          std::tr1::tuple<PrefHashStoreTransaction::ValueState, bool>> {
  protected:
   void SetUp() override {
     PreferenceValidationDelegateTest::SetUp();
     value_state_ = std::tr1::get<0>(GetParam());
-    external_validation_value_state_ = std::tr1::get<1>(GetParam());
-    is_personal_ = std::tr1::get<2>(GetParam());
+    is_personal_ = std::tr1::get<1>(GetParam());
   }
 
   PrefHashStoreTransaction::ValueState value_state_;
-  PrefHashStoreTransaction::ValueState external_validation_value_state_;
   bool is_personal_;
 };
 
 TEST_P(PreferenceValidationDelegateWithIncident, Atomic) {
   instance_->OnAtomicPreferenceValidation(
-      kPrefPath_, null_value_.get(), value_state_,
-      external_validation_value_state_, is_personal_);
+      kPrefPath_, null_value_.get(), value_state_, is_personal_);
   ASSERT_EQ(1U, incidents_.size());
   std::unique_ptr<safe_browsing::ClientIncidentReport_IncidentData> incident(
       incidents_.back()->TakePayload());
   EXPECT_TRUE(incident->has_tracked_preference());
   const safe_browsing::
       ClientIncidentReport_IncidentData_TrackedPreferenceIncident& tp_incident =
           incident->tracked_preference();
   EXPECT_EQ(kPrefPath_, tp_incident.path());
   EXPECT_EQ(0, tp_incident.split_key_size());
   if (!is_personal_) {
     EXPECT_TRUE(tp_incident.has_atomic_value());
     EXPECT_EQ(std::string("null"), tp_incident.atomic_value());
   } else {
     EXPECT_FALSE(tp_incident.has_atomic_value());
   }
   EXPECT_TRUE(tp_incident.has_value_state());
-  ExpectValueStatesEquate(value_state_, external_validation_value_state_,
-                          tp_incident.value_state());
+  ExpectValueStatesEquate(value_state_, tp_incident.value_state());
 }
 
 TEST_P(PreferenceValidationDelegateWithIncident, Split) {
   instance_->OnSplitPreferenceValidation(
-      kPrefPath_, &dict_value_, invalid_keys_, value_state_,
-      external_validation_value_state_, is_personal_);
+      kPrefPath_, &dict_value_, invalid_keys_, value_state_, is_personal_);
   ASSERT_EQ(1U, incidents_.size());
   std::unique_ptr<safe_browsing::ClientIncidentReport_IncidentData> incident(
       incidents_.back()->TakePayload());
   EXPECT_TRUE(incident->has_tracked_preference());
   const safe_browsing::
       ClientIncidentReport_IncidentData_TrackedPreferenceIncident& tp_incident =
           incident->tracked_preference();
   EXPECT_EQ(kPrefPath_, tp_incident.path());
   EXPECT_FALSE(tp_incident.has_atomic_value());
   if (!is_personal_)
     ExpectKeysEquate(invalid_keys_, tp_incident.split_key());
   else
     EXPECT_EQ(0, tp_incident.split_key_size());
   EXPECT_TRUE(tp_incident.has_value_state());
-  ExpectValueStatesEquate(value_state_, external_validation_value_state_,
-                          tp_incident.value_state());
+  ExpectValueStatesEquate(value_state_, tp_incident.value_state());
 }
 
 INSTANTIATE_TEST_CASE_P(
     WithIncident,
     PreferenceValidationDelegateWithIncident,
     testing::Combine(
         testing::Values(PrefHashStoreTransaction::CLEARED,
                         PrefHashStoreTransaction::CHANGED,
                         PrefHashStoreTransaction::UNTRUSTED_UNKNOWN_VALUE),
-        testing::Values(PrefHashStoreTransaction::UNCHANGED,
-                        PrefHashStoreTransaction::UNSUPPORTED,
-                        PrefHashStoreTransaction::UNTRUSTED_UNKNOWN_VALUE),
         testing::Bool()));
-
-INSTANTIATE_TEST_CASE_P(
-    WithBypassIncident,
-    PreferenceValidationDelegateWithIncident,
-    testing::Combine(
-        testing::Values(PrefHashStoreTransaction::UNCHANGED,
-                        PrefHashStoreTransaction::SECURE_LEGACY,
-                        PrefHashStoreTransaction::TRUSTED_UNKNOWN_VALUE),
-        testing::Values(PrefHashStoreTransaction::CHANGED,
-                        PrefHashStoreTransaction::CLEARED),
-        testing::Bool()));