Remove initial_sync_ended bits

sync/internal_api/public/sessions/sync_session_snapshot.h

Index: sync/internal_api/public/sessions/sync_session_snapshot.h
diff --git a/sync/internal_api/public/sessions/sync_session_snapshot.h b/sync/internal_api/public/sessions/sync_session_snapshot.h
index 82524198c3e1f589183e2fd24c91b96a02670c0e..d13f03923813bce9f7397be899cf654c2f71e03f 100644
--- a/sync/internal_api/public/sessions/sync_session_snapshot.h
+++ b/sync/internal_api/public/sessions/sync_session_snapshot.h
@@ -31,8 +31,6 @@ class SyncSessionSnapshot {
   SyncSessionSnapshot();
   SyncSessionSnapshot(
       const ModelNeutralState& model_neutral_state,
-      bool is_share_usable,

[Nicolas Zea, 2012/12/10 22:50:38]

why is is_share_usable being removed as well? (I'll be honest, I don't even know
what it's for)

[tim (not reviewing), 2012/12/10 23:05:36]

It used to denote "initial sync ended for all datatypes", and we wouldn't let
anything interact with the sync engine until that condition was met.  The
concept doesn't really make sense anymore though and AFAICT it's not used in any
important way. I think AllStatus uses it, but at the time is_share_usable was
relevant AllStatus wasn't, so I doubt it's doing anything important.

-      ModelTypeSet initial_sync_ended,
       const ProgressMarkerMap& download_progress_markers,
       bool is_silenced,
       int num_encryption_conflicts,
@@ -56,8 +54,6 @@ class SyncSessionSnapshot {
     return model_neutral_state_;
   }
   int64 num_server_changes_remaining() const;
-  bool is_share_usable() const;
-  ModelTypeSet initial_sync_ended() const;
   const ProgressMarkerMap& download_progress_markers() const;
   bool is_silenced() const;
   int num_encryption_conflicts() const;
@@ -76,8 +72,6 @@ class SyncSessionSnapshot {
 
  private:
   ModelNeutralState model_neutral_state_;
-  bool is_share_usable_;
-  ModelTypeSet initial_sync_ended_;
   ProgressMarkerMap download_progress_markers_;
   bool is_silenced_;
   int num_encryption_conflicts_;