sync: GU retry with less explicit TimeTicks logic

sync/sessions/nudge_tracker.h

 // Copyright 2013 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.
 //
 // A class to track the outstanding work required to bring the client back into
 // sync with the server.
 #ifndef SYNC_SESSIONS_NUDGE_TRACKER_H_
 #define SYNC_SESSIONS_NUDGE_TRACKER_H_
 
 #include <list>
@@ skipping 18 matching lines @@
   NudgeTracker();
   ~NudgeTracker();
 
   // Returns true if there is a good reason for performing a sync cycle.
   // This does not take into account whether or not this is a good *time* to
   // perform a sync cycle; that's the scheduler's job.
   bool IsSyncRequired() const;
 
   // Returns true if there is a good reason for performing a get updates
   // request as part of the next sync cycle.
-  bool IsGetUpdatesRequired(base::TimeTicks now) const;
+  bool IsGetUpdatesRequired() const;
 
   // Return true if should perform a sync cycle for GU retry.
-  bool IsRetryRequired(base::TimeTicks now) const;
+  //
+  // This is sensitive to changes in 'current time'.  Make sure you're aware of
+  // any nearby calls to ToggleRetryFlagIfRequired().
+  bool IsRetryRequired() const;
 
   // Tells this class that all required update fetching or committing has
   // completed successfully.
-  void RecordSuccessfulSyncCycle(base::TimeTicks now);
+  void RecordSuccessfulSyncCycle();
 
   // Takes note of a local change.
   void RecordLocalChange(ModelTypeSet types);
 
   // Takes note of a locally issued request to refresh a data type.
   void RecordLocalRefreshRequest(ModelTypeSet types);
 
   // Takes note of the receipt of an invalidation notice from the server.
   void RecordRemoteInvalidation(
       const ObjectIdInvalidationMap& invalidation_map);
@@ skipping 41 matching lines @@
       ModelType type,
       sync_pb::GetUpdateTriggers* msg) const;
 
   // Fills a ProgressMarker with single legacy notification hint expected by the
   // sync server.  Newer servers will rely on the data set by FillProtoMessage()
   // instead of this.
   void SetLegacyNotificationHint(
       ModelType type,
       sync_pb::DataTypeProgressMarker* progress) const;
 
+  // Flips the flag if we're due for a retry.
+  void ToggleRetryFlagIfRequired(base::TimeTicks now);
+
   // Adjusts the number of hints that can be stored locally.
   void SetHintBufferSize(size_t size);
 
-  void set_next_retry_time(base::TimeTicks next_retry_time) {
-    next_retry_time_ = next_retry_time;
-  }
+  // Schedules a retry GetUpdate request for some time in the future.
+  //
+  // The effects of this change are not guaranteed to take effect until
+  // ToggleRetryFlagIfRequired() is called.  That function should be called
+  // rarely (ie. only before a sync cycle, never during).
+  void SetNextRetryTime(base::TimeTicks next_retry_time);
 
  private:
   typedef std::map<ModelType, DataTypeTracker> TypeTrackerMap;
 
   TypeTrackerMap type_trackers_;
 
   // Merged updates source.  This should be obsolete, but the server still
   // relies on it for some heuristics.
   sync_pb::GetUpdatesCallerInfo::GetUpdatesSource updates_source_;
 
   // Tracks whether or not invalidations are currently enabled.
   bool invalidations_enabled_;
 
   // This flag is set if suspect that some technical malfunction or known bug
   // may have left us with some unserviced invalidations.
   //
   // Keeps track of whether or not we're fully in sync with the invalidation
   // server.  This can be false even if invalidations are enabled and working
   // correctly.  For example, until we get ack-tracking working properly, we
   // won't persist invalidations between restarts, so we may be out of sync when
   // we restart.  The only way to get back into sync is to have invalidations
   // enabled, then complete a sync cycle to make sure we're fully up to date.
   bool invalidations_out_of_sync_;
 
   size_t num_payloads_per_type_;
 
   base::TimeTicks last_successful_sync_time_;
 
-  // A retry GU should be issued after this time.
+  // Time when the next retry is scheduled to execute.  Will be null if no
+  // retry is currently pending, or the is_retry_required flag is set.
   base::TimeTicks next_retry_time_;
 
+  // Set to true when the nudge tracker believes we're due for a GU retry.
+  bool is_retry_required_;
+
   DISALLOW_COPY_AND_ASSIGN(NudgeTracker);
 };
 
 }  // namespace sessions
 }  // namespace syncer
 
 #endif  // SYNC_SESSIONS_NUDGE_TRACKER_H_