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

Unified Diff: components/browsing_data/core/counters/history_counter.cc

Issue 2855623005: Move browsing_data counter sync tests to separate file (Closed)
Patch Set: remove HistoryResult and PasswordResult Created 3 years, 7 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: components/browsing_data/core/counters/history_counter.cc
diff --git a/components/browsing_data/core/counters/history_counter.cc b/components/browsing_data/core/counters/history_counter.cc
index 6fcb1c17ea94ec4a88a6225c190ccf4e7fd2ce78..ee93c1265d516e13bb4c1955e6cbd3914c3e0744 100644
--- a/components/browsing_data/core/counters/history_counter.cc
+++ b/components/browsing_data/core/counters/history_counter.cc
@@ -150,16 +150,9 @@ void HistoryCounter::MergeResults() {
return;
ReportResult(
- base::MakeUnique<HistoryResult>(this, local_result_, has_synced_visits_));
+ base::MakeUnique<SyncResult>(this, local_result_, has_synced_visits_));
}
-HistoryCounter::HistoryResult::HistoryResult(const HistoryCounter* source,
- ResultInt value,
- bool has_synced_visits)
- : FinishedResult(source, value), has_synced_visits_(has_synced_visits) {}
-
-HistoryCounter::HistoryResult::~HistoryResult() {}
-
void HistoryCounter::OnStateChanged(syncer::SyncService* sync) {
bool history_sync_enabled_new_state = !!web_history_service_callback_.Run();

Powered by Google App Engine
This is Rietveld 408576698