sync: use progress markers instead of timestamps during GetUpdates

chrome/browser/sync/sessions/status_controller_unittest.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/sessions/sync_session.h"
 #include "chrome/test/sync/engine/test_id_factory.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace browser_sync {
 namespace sessions {
@@ skipping 29 matching lines @@
   EXPECT_TRUE(status.TestAndClearIsDirty());
   status.set_num_consecutive_errors(10);
   EXPECT_FALSE(status.TestAndClearIsDirty());  // Only dirty if value changed.
   status.increment_num_consecutive_errors();
   EXPECT_TRUE(status.TestAndClearIsDirty());
   status.increment_num_consecutive_errors_by(1);
   EXPECT_TRUE(status.TestAndClearIsDirty());
   status.increment_num_consecutive_errors_by(0);
   EXPECT_FALSE(status.TestAndClearIsDirty());
 
-  {
-    ScopedModelSafeGroupRestriction r(&status, GROUP_UI);
-    status.set_current_download_timestamp(syncable::BOOKMARKS, 100);
-    EXPECT_TRUE(status.TestAndClearIsDirty());
-  }
-
   status.set_num_server_changes_remaining(30);
   EXPECT_TRUE(status.TestAndClearIsDirty());
 
   status.set_invalid_store(true);
   EXPECT_TRUE(status.TestAndClearIsDirty());
   status.set_invalid_store(false);
   EXPECT_TRUE(status.TestAndClearIsDirty());
 
   status.set_syncer_stuck(true);
   EXPECT_TRUE(status.TestAndClearIsDirty());
@@ skipping 57 matching lines @@
   status.increment_num_consecutive_transient_error_commits_by(0);
   EXPECT_EQ(7, status.error_counters().consecutive_transient_error_commits);
 
   status.set_num_consecutive_errors(8);
   EXPECT_EQ(8, status.error_counters().consecutive_errors);
   status.increment_num_consecutive_errors();
   EXPECT_EQ(9, status.error_counters().consecutive_errors);
   status.increment_num_consecutive_errors_by(2);
   EXPECT_EQ(11, status.error_counters().consecutive_errors);
 
-  {
-    ScopedModelSafeGroupRestriction r(&status, GROUP_UI);
-    status.set_current_download_timestamp(syncable::BOOKMARKS, 12);
-    EXPECT_EQ(12, status.ComputeMaxLocalTimestamp());
-  }
-
   status.set_num_server_changes_remaining(13);
   EXPECT_EQ(13, status.num_server_changes_remaining());
 
   EXPECT_FALSE(status.syncer_status().invalid_store);
   status.set_invalid_store(true);
   EXPECT_TRUE(status.syncer_status().invalid_store);
 
   EXPECT_FALSE(status.syncer_status().syncer_stuck);
   status.set_syncer_stuck(true);
   EXPECT_TRUE(status.syncer_status().syncer_stuck);
@@ skipping 70 matching lines @@
   status.GetUnrestrictedUpdateProgress(
       GROUP_UI)->SuccessfullyAppliedUpdateCount();
   status.mutable_commit_message();
   status.commit_response();
   status.mutable_commit_response();
   status.updates_response();
   status.mutable_updates_response();
   status.error_counters();
   status.syncer_status();
   status.num_server_changes_remaining();
-  status.ComputeMaxLocalTimestamp();
   status.commit_ids();
   status.HasBookmarkCommitActivity();
   status.download_updates_succeeded();
   status.ServerSaysNothingMoreToDownload();
   status.group_restriction();
 }
 
 }  // namespace sessions
 }  // namespace browser_sync