[Sync] - In about:sync page display if the first sync after restart has taken place or not.

chrome/browser/sync/internal_api/sync_manager.h

 // 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.
 
 #ifndef CHROME_BROWSER_SYNC_INTERNAL_API_SYNC_MANAGER_H_
 #define CHROME_BROWSER_SYNC_INTERNAL_API_SYNC_MANAGER_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 76 matching lines @@
   // Status encapsulates detailed state about the internals of the SyncManager.
   struct Status {
     // Summary is a distilled set of important information that the end-user may
     // wish to be informed about (through UI, for example). Note that if a
     // summary state requires user interaction (such as auth failures), more
     // detailed information may be contained in additional status fields.
     enum Summary {
       // The internal instance is in an unrecognizable state. This should not
       // happen.
       INVALID = 0,
+      // A sync has not happened since restart.
+      UNINITIALIZED,
       // Can't connect to server, but there are no pending changes in
       // our local cache.
       OFFLINE,
       // Can't connect to server, and there are pending changes in our
       // local cache.
       OFFLINE_UNSYNCED,
       // Connected and syncing.
       SYNCING,
       // Connected, no pending changes.
       READY,
-      // Can't connect to server, and we haven't completed the initial
-      // sync yet.  So there's nothing we can do but wait for the server.
+      // Can't connect to server.

[rlarocque, 2012/02/03 18:55:03]

If we're going to change the meaning of an enum, we should change its name, too,
just to avoid confusion.  Otherwise we'll have to go digging through commit logs
to figure out what "OFFLINE_UNUSABLE" actually means for a given revision number
every time that status shows up on crbug.

Regardless, I don't think we should be making this change in a commit that is
meant to fix a different bug and is headed for M18.  

See also: comment in all_status.cc

[lipalani1, 2012/02/07 23:34:11]

Done.

+      // The state indicates there are no pending changes but
+      // the client is possibly trying to sync because of a notification.
       OFFLINE_UNUSABLE,
 
       SUMMARY_STATUS_COUNT,
     };
 
     Status();
     ~Status();
 
     Summary summary;
     bool authenticated;      // Successfully authenticated via GAIA.
@@ skipping 30 matching lines @@
     int tombstone_updates_received;
 
     // Total number of overwrites due to conflict resolver since browser start.
     int num_local_overwrites_total;
     int num_server_overwrites_total;
 
     // Count of empty and non empty getupdates;
     int nonempty_get_updates;
     int empty_get_updates;
 
+    // Number of sync cycles.
+    int sync_count;
+
     // Count of useless and useful syncs we perform.
     int useless_sync_cycles;
     int useful_sync_cycles;
 
     // Encryption related.
     syncable::ModelTypeSet encrypted_types;
     bool cryptographer_ready;
     bool crypto_has_pending_keys;
 
     // The unique identifer for this client.
@@ skipping 448 matching lines @@
 syncable::ModelTypeSet GetTypesWithEmptyProgressMarkerToken(
     syncable::ModelTypeSet types,
     sync_api::UserShare* share);
 
 // Returns the string representation of a PassphraseRequiredReason value.
 std::string PassphraseRequiredReasonToString(PassphraseRequiredReason reason);
 
 }  // namespace sync_api
 
 #endif  // CHROME_BROWSER_SYNC_INTERNAL_API_SYNC_MANAGER_H_