Make it clear what last_sync_timestamp actually tracks. Update

chrome/browser/sync/engine/verify_updates_command.cc

 // Copyright (c) 2009 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/engine/verify_updates_command.h"
 
 #include "chrome/browser/sync/engine/syncer.h"
 #include "chrome/browser/sync/engine/syncer_proto_util.h"
 #include "chrome/browser/sync/engine/syncer_types.h"
@@ skipping 24 matching lines @@
     LOG(ERROR) << "Scoped dir lookup failed!";
     return;
   }
   WriteTransaction trans(dir, SYNCER, __FILE__, __LINE__);
   sessions::StatusController* status = session->status_controller();
   const GetUpdatesResponse& updates = status->updates_response().get_updates();
   int update_count = updates.entries().size();
 
   LOG(INFO) << update_count << " entries to verify";
   for (int i = 0; i < update_count; i++) {
-    const SyncEntity entry =
+    const SyncEntity& entry =
         *reinterpret_cast<const SyncEntity *>(&(updates.entries(i)));
     ModelSafeGroup g = GetGroupForModelType(entry.GetModelType(),
                                             session->routing_info());
     if (g != status->group_restriction())
       continue;
 
     // Needs to be done separately in order to make sure the update processing
     // still happens like normal. We should really just use one type of
     // ID in fact, there isn't actually a need for server_knows and not IDs.
     SyncerUtil::AttemptReuniteLostCommitResponses(&trans, entry,
@@ skipping 73 matching lines @@
         deleted, is_directory, model_type);
   }
 
   if (VERIFY_UNDECIDED == result.value)
     result.value = VERIFY_SUCCESS;  // No news is good news.
 
   return result;  // This might be VERIFY_SUCCESS as well
 }
 
 }  // namespace browser_sync