[Sync] Convert SyncSessionSnapshot to a copy-able class.

chrome/browser/sync/profile_sync_service_harness.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_PROFILE_SYNC_SERVICE_HARNESS_H_
 #define CHROME_BROWSER_SYNC_PROFILE_SYNC_SERVICE_HARNESS_H_
 #pragma once
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "chrome/browser/sync/backend_migrator.h"
 #include "chrome/browser/sync/profile_sync_service.h"
 #include "chrome/browser/sync/profile_sync_service_observer.h"
 #include "chrome/browser/sync/retry_verifier.h"
 #include "sync/syncable/model_type.h"
 
 class Profile;
 
 namespace browser_sync {
-  namespace sessions {
-    struct SyncSessionSnapshot;
-  }
+namespace sessions {
+class SyncSessionSnapshot;
+}
 }
 
 // An instance of this class is basically our notion of a "sync client" for
 // automation purposes. It harnesses the ProfileSyncService member of the
 // profile passed to it on construction and automates certain things like setup
 // and authentication. It provides ways to "wait" adequate periods of time for
 // several clients to get to the same state.
 class ProfileSyncServiceHarness
     : public ProfileSyncServiceObserver,
       public browser_sync::MigrationObserver {
@@ skipping 118 matching lines @@
   // Disables sync for a particular sync datatype. Returns true on success.
   bool DisableSyncForDatatype(syncable::ModelType datatype);
 
   // Enables sync for all sync datatypes. Returns true on success.
   bool EnableSyncForAllDatatypes();
 
   // Disables sync for all sync datatypes. Returns true on success.
   bool DisableSyncForAllDatatypes();
 
   // Returns a snapshot of the current sync session.
-  const browser_sync::sessions::SyncSessionSnapshot*
-      GetLastSessionSnapshot() const;
+  browser_sync::sessions::SyncSessionSnapshot GetLastSessionSnapshot() const;
 
   // Encrypt the datatype |type|. This method will block while the sync backend
   // host performs the encryption or a timeout is reached.
   // PostCondition:
   //   returns: True if |type| was encrypted and we are fully synced.
   //            False if we timed out.
   bool EnableEncryptionForType(syncable::ModelType type);
 
   // Wait until |type| is encrypted or we time out.
   // PostCondition:
@@ skipping 107 matching lines @@
 
   // Finite state machine for controlling state.  Returns true only if a state
   // change has taken place.
   bool RunStateChangeMachine();
 
   // Returns true if a status change took place, false on timeout.
   bool AwaitStatusChangeWithTimeout(int timeout_milliseconds,
                                     const std::string& reason);
 
   // A helper for implementing IsDataSynced() and IsFullySynced().
-  bool IsDataSyncedImpl(const browser_sync::sessions::SyncSessionSnapshot*);
+  bool IsDataSyncedImpl(
+      const browser_sync::sessions::SyncSessionSnapshot& snapshot);
 
   // Returns true if the sync client has no unsynced items.
   bool IsDataSynced();
 
   // Returns true if the sync client has no unsynced items and its progress
   // markers are believed to be up to date.
   //
   // Although we can't detect when commits from other clients invalidate our
   // local progress markers, we do know when our own commits have invalidated
   // our timestmaps.  This check returns true when this client has, to the best
@@ skipping 52 matching lines @@
   const std::string profile_debug_name_;
 
   // Keeps track of the number of attempts at exponential backoff and its
   // related bookkeeping information for verification.
   browser_sync::RetryVerifier retry_verifier_;
 
   DISALLOW_COPY_AND_ASSIGN(ProfileSyncServiceHarness);
 };
 
 #endif  // CHROME_BROWSER_SYNC_PROFILE_SYNC_SERVICE_HARNESS_H_