Sync: Improve handling of database load failures

chrome/browser/sync/glue/sync_backend_host.h

 // Copyright (c) 2011 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_GLUE_SYNC_BACKEND_HOST_H_
 #define CHROME_BROWSER_SYNC_GLUE_SYNC_BACKEND_HOST_H_
 #pragma once
 
 #include <map>
 #include <string>
@@ skipping 165 matching lines @@
   // using their Google Account password.  An implicit SetPassphrase will *not*
   // *not* override an explicit passphrase set previously.
   void SetPassphrase(const std::string& passphrase, bool is_explicit);
 
   // Called on |frontend_loop_| to kick off shutdown procedure. After this,
   // no further sync activity will occur with the sync server and no further
   // change applications will occur from changes already downloaded.
   virtual void StopSyncingForShutdown();
 
   // Called on |frontend_loop_| to kick off shutdown.
-  // |sync_disabled| indicates if syncing is being disabled or not.
-  // See the implementation and Core::DoShutdown for details.
-  // Must be called *after* StopSyncingForShutdown.
-  void Shutdown(bool sync_disabled);
+  // |delete_stale_data| indicates whether or not we should delete the local
+  // sync database.
+  // See the implementation and Core::DoShutdown for details.  Must be called
+  // *after* StopSyncingForShutdown.
+  void Shutdown(bool delete_stale_data);
 
   // Changes the set of data types that are currently being synced.
   // The ready_task will be run when all of the requested data types
   // are up-to-date and ready for activation.  The task will be
   // cancelled upon shutdown.
   virtual void ConfigureDataTypes(
       const syncable::ModelTypeSet& types_to_add,
       const syncable::ModelTypeSet& types_to_remove,
       sync_api::ConfigureReason reason,
       base::Callback<void(bool)> ready_task,
@@ skipping 181 matching lines @@
     // 1) From |sync_thread_|, invoke the syncapi Shutdown call to do
     //    a final SaveChanges, and close sqlite handles.
     // 2) Then, from |frontend_loop_|, halt the sync_thread_ (which is
     //    a blocking call). This causes syncapi thread-exit handlers
     //    to run and make use of cached pointers to various components
     //    owned implicitly by us.
     // 3) Destroy this Core. That will delete syncapi components in a
     //    safe order because the thread that was using them has exited
     //    (in step 2).
     void DoStopSyncManagerForShutdown(const base::Closure& closure);
-    void DoShutdown(bool stopping_sync);
+    void DoShutdown(bool delete_stale_data);
 
     virtual void DoRequestConfig(
         const syncable::ModelTypeBitSet& types_to_config,
         sync_api::ConfigureReason reason);
 
     // Start the configuration mode.  |callback| is called on the sync
     // thread.
     virtual void DoStartConfiguration(const base::Closure& callback);
 
     // Set the base request context to use when making HTTP calls.
@@ skipping 196 matching lines @@
 
   // UI-thread cache of the last SyncSessionSnapshot received from syncapi.
   scoped_ptr<sessions::SyncSessionSnapshot> last_snapshot_;
 
   DISALLOW_COPY_AND_ASSIGN(SyncBackendHost);
 };
 
 }  // namespace browser_sync
 
 #endif  // CHROME_BROWSER_SYNC_GLUE_SYNC_BACKEND_HOST_H_