Correctly handle expired and truncated visits in typed url sync

chrome/browser/sync/glue/sync_backend_registrar.cc

 // Copyright (c) 2011 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/sync/glue/sync_backend_registrar.h"
 
 #include <algorithm>
 #include <cstddef>
 
 #include "base/compiler_specific.h"
@@ skipping 105 matching lines @@
                           types_to_remove.begin(), types_to_remove.end(),
                           std::inserter(intersection, intersection.end()));
     DCHECK(intersection.empty());
   }
   syncable::ModelTypeSet filtered_types_to_add = types_to_add;
   if (workers_.count(GROUP_HISTORY) == 0) {
     LOG(WARNING) << "No history worker -- removing TYPED_URLS";
     filtered_types_to_add.erase(syncable::TYPED_URLS);
   }
   if (workers_.count(GROUP_PASSWORD) == 0) {
-    LOG(WARNING) << "No history worker -- removing PASSWORDS";
+    LOG(WARNING) << "No password worker -- removing PASSWORDS";

[Andrew T Wilson (Slow), 2011/10/21 13:14:05]

This copy/paste error keeps bugging me, so I fixed it to be more correct.

     filtered_types_to_add.erase(syncable::PASSWORDS);
   }
 
   base::AutoLock lock(lock_);
   syncable::ModelTypeSet newly_added_types;
   for (syncable::ModelTypeSet::const_iterator it =
            filtered_types_to_add.begin();
        it != filtered_types_to_add.end(); ++it) {
     // Add a newly specified data type as GROUP_PASSIVE into the
     // routing_info, if it does not already exist.
@@ skipping 145 matching lines @@
   return it->second;
 }
 
 bool SyncBackendRegistrar::IsCurrentThreadSafeForModel(
     syncable::ModelType model_type) const {
   lock_.AssertAcquired();
   return IsOnThreadForGroup(GetGroupForModelType(model_type, routing_info_));
 }
 
 }  // namespace browser_sync
-