sync: Follow-up to conflict resolution refactor

sync/sessions/status_controller.cc

 // Copyright (c) 2012 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 "sync/sessions/status_controller.h"
 
 #include <vector>
 
 #include "base/basictypes.h"
 #include "sync/internal_api/public/base/model_type.h"
 #include "sync/protocol/sync_protocol_error.h"
 
 namespace syncer {
 namespace sessions {
 
 StatusController::StatusController(const ModelSafeRoutingInfo& routes)
-    : per_model_group_deleter_(&per_model_group_),
-      group_restriction_in_effect_(false),
+    : group_restriction_in_effect_(false),
       group_restriction_(GROUP_PASSIVE),
       routing_info_(routes) {
 }
 
 StatusController::~StatusController() {}
 
-const std::set<syncable::Id>* StatusController::simple_conflict_ids() const {
-  const PerModelSafeGroupState* state =
-      GetModelSafeGroupState(true, group_restriction_);
-  return state ? &state->simple_conflict_ids : NULL;
-}
-
-std::set<syncable::Id>* StatusController::mutable_simple_conflict_ids() {
-  return &GetOrCreateModelSafeGroupState(
-      true, group_restriction_)->simple_conflict_ids;
-}
-
-const std::set<syncable::Id>*
-    StatusController::GetUnrestrictedSimpleConflictIds(
-        ModelSafeGroup group) const {
-  const PerModelSafeGroupState* state = GetModelSafeGroupState(false, group);
-  return state ? &state->simple_conflict_ids : NULL;
-}
-
-const PerModelSafeGroupState* StatusController::GetModelSafeGroupState(
-    bool restrict, ModelSafeGroup group) const {
-  DCHECK_EQ(restrict, group_restriction_in_effect_);
-  std::map<ModelSafeGroup, PerModelSafeGroupState*>::const_iterator it =
-      per_model_group_.find(group);
-  return (it == per_model_group_.end()) ? NULL : it->second;
-}
-
-PerModelSafeGroupState* StatusController::GetOrCreateModelSafeGroupState(
-    bool restrict, ModelSafeGroup group) {
-  DCHECK_EQ(restrict, group_restriction_in_effect_);
-  std::map<ModelSafeGroup, PerModelSafeGroupState*>::iterator it =
-      per_model_group_.find(group);
-  if (it == per_model_group_.end()) {
-    PerModelSafeGroupState* state = new PerModelSafeGroupState();
-    it = per_model_group_.insert(std::make_pair(group, state)).first;
-  }
-  return it->second;
-}
-
 void StatusController::increment_num_updates_downloaded_by(int value) {
   model_neutral_.num_updates_downloaded_total += value;
 }
 
 void StatusController::set_types_needing_local_migration(ModelTypeSet types) {
   model_neutral_.types_needing_local_migration = types;
 }
 
 void StatusController::increment_num_tombstone_updates_downloaded_by(
     int value) {
@@ skipping 65 matching lines @@
 }
 
 void StatusController::set_commit_result(const SyncerError result) {
   model_neutral_.commit_result = result;
 }
 
 SyncerError StatusController::last_get_key_result() const {
   return model_neutral_.last_get_key_result;
 }
 
-void StatusController::update_conflicts_resolved(bool resolved) {
-  model_neutral_.conflicts_resolved |= resolved;
-}
-void StatusController::reset_conflicts_resolved() {
-  model_neutral_.conflicts_resolved = false;
-}
-
 // Returns the number of updates received from the sync server.
 int64 StatusController::CountUpdates() const {
   const sync_pb::ClientToServerResponse& updates =
       model_neutral_.updates_response;
   if (updates.has_get_updates()) {
     return updates.get_updates().entries().size();
   } else {
     return 0;
   }
 }
 
-bool StatusController::HasConflictingUpdates() const {
-  return TotalNumConflictingItems() > 0;
-}
-
 int StatusController::num_updates_applied() const {
   return model_neutral_.num_updates_applied;
 }
 
 int StatusController::num_server_overwrites() const {
   return model_neutral_.num_server_overwrites;
 }
 
 int StatusController::num_encryption_conflicts() const {
   return model_neutral_.num_encryption_conflicts;
 }
 
 int StatusController::num_hierarchy_conflicts() const {
   DCHECK(!group_restriction_in_effect_)
       << "num_hierarchy_conflicts applies to all ModelSafeGroups";
   return model_neutral_.num_hierarchy_conflicts;
 }
 
-int StatusController::num_simple_conflicts() const {
-  DCHECK(!group_restriction_in_effect_)
-   << "num_simple_conflicts applies to all ModelSafeGroups";
-  std::map<ModelSafeGroup, PerModelSafeGroupState*>::const_iterator it =
-      per_model_group_.begin();
-  int sum = 0;
-  for (; it != per_model_group_.end(); ++it) {
-    sum += it->second->simple_conflict_ids.size();
-  }
-  return sum;
-}
-
 int StatusController::num_server_conflicts() const {
   DCHECK(!group_restriction_in_effect_)
       << "num_server_conflicts applies to all ModelSafeGroups";
   return model_neutral_.num_server_conflicts;
 }
 
 int StatusController::TotalNumConflictingItems() const {
   DCHECK(!group_restriction_in_effect_)
       << "TotalNumConflictingItems applies to all ModelSafeGroups";
   int sum = 0;
-  sum += num_simple_conflicts();
   sum += num_encryption_conflicts();
   sum += num_hierarchy_conflicts();
   sum += num_server_conflicts();
   return sum;
 }
 
 bool StatusController::ServerSaysNothingMoreToDownload() const {
   if (!download_updates_succeeded())
     return false;
 
   if (!updates_response().get_updates().has_changes_remaining()) {
     NOTREACHED();  // Server should always send changes remaining.
     return false;  // Avoid looping forever.
   }
   // Changes remaining is an estimate, but if it's estimated to be
   // zero, that's firm and we don't have to ask again.
   return updates_response().get_updates().changes_remaining() == 0;
 }
 
 void StatusController::set_debug_info_sent() {
   model_neutral_.debug_info_sent = true;
 }
 
 bool StatusController::debug_info_sent() const {
   return model_neutral_.debug_info_sent;
 }
 
 }  // namespace sessions
 }  // namespace syncer