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

Unified Diff: chrome/browser/safe_browsing/preference_validation_delegate.cc

Issue 470213002: Moving all the incident reporting stuff to a common folder (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@binsign
Patch Set: Created 6 years, 4 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/preference_validation_delegate.cc
diff --git a/chrome/browser/safe_browsing/preference_validation_delegate.cc b/chrome/browser/safe_browsing/preference_validation_delegate.cc
deleted file mode 100644
index 55ad2a549be76b75298639cd64986ceb661d5106..0000000000000000000000000000000000000000
--- a/chrome/browser/safe_browsing/preference_validation_delegate.cc
+++ /dev/null
@@ -1,93 +0,0 @@
-// 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/preference_validation_delegate.h"
-
-#include <string>
-#include <vector>
-
-#include "base/callback.h"
-#include "base/json/json_writer.h"
-#include "chrome/browser/prefs/pref_hash_store_transaction.h"
-#include "chrome/browser/prefs/tracked/tracked_preference_helper.h"
-#include "chrome/common/safe_browsing/csd.pb.h"
-
-namespace safe_browsing {
-
-namespace {
-
-typedef ClientIncidentReport_IncidentData_TrackedPreferenceIncident TPIncident;
-typedef ClientIncidentReport_IncidentData_TrackedPreferenceIncident_ValueState
- TPIncident_ValueState;
-
-// Maps a PrefHashStoreTransaction::ValueState to a
-// TrackedPreferenceIncident::ValueState.
-TPIncident_ValueState MapValueState(
- PrefHashStoreTransaction::ValueState value_state) {
- switch (value_state) {
- case PrefHashStoreTransaction::CLEARED:
- return TPIncident::CLEARED;
- case PrefHashStoreTransaction::CHANGED:
- return TPIncident::CHANGED;
- case PrefHashStoreTransaction::UNTRUSTED_UNKNOWN_VALUE:
- return TPIncident::UNTRUSTED_UNKNOWN_VALUE;
- default:
- return TPIncident::UNKNOWN;
- }
-}
-
-} // namespace
-
-PreferenceValidationDelegate::PreferenceValidationDelegate(
- const AddIncidentCallback& add_incident)
- : add_incident_(add_incident) {
-}
-
-PreferenceValidationDelegate::~PreferenceValidationDelegate() {
-}
-
-void PreferenceValidationDelegate::OnAtomicPreferenceValidation(
- const std::string& pref_path,
- const base::Value* value,
- PrefHashStoreTransaction::ValueState value_state,
- TrackedPreferenceHelper::ResetAction /* reset_action */) {
- TPIncident_ValueState proto_value_state = MapValueState(value_state);
- if (proto_value_state != TPIncident::UNKNOWN) {
- scoped_ptr<ClientIncidentReport_IncidentData> incident_data(
- new ClientIncidentReport_IncidentData());
- TPIncident* incident = incident_data->mutable_tracked_preference();
- incident->set_path(pref_path);
- if (!value ||
- (!value->GetAsString(incident->mutable_atomic_value()) &&
- !base::JSONWriter::Write(value, incident->mutable_atomic_value()))) {
- incident->clear_atomic_value();
- }
- incident->set_value_state(proto_value_state);
- add_incident_.Run(incident_data.Pass());
- }
-}
-
-void PreferenceValidationDelegate::OnSplitPreferenceValidation(
- const std::string& pref_path,
- const base::DictionaryValue* /* dict_value */,
- const std::vector<std::string>& invalid_keys,
- PrefHashStoreTransaction::ValueState value_state,
- TrackedPreferenceHelper::ResetAction /* reset_action */) {
- TPIncident_ValueState proto_value_state = MapValueState(value_state);
- if (proto_value_state != TPIncident::UNKNOWN) {
- scoped_ptr<ClientIncidentReport_IncidentData> incident_data(
- new ClientIncidentReport_IncidentData());
- TPIncident* incident = incident_data->mutable_tracked_preference();
- incident->set_path(pref_path);
- for (std::vector<std::string>::const_iterator scan(invalid_keys.begin());
- scan != invalid_keys.end();
- ++scan) {
- incident->add_split_key(*scan);
- }
- incident->set_value_state(proto_value_state);
- add_incident_.Run(incident_data.Pass());
- }
-}
-
-} // namespace safe_browsing

Powered by Google App Engine
This is Rietveld 408576698