String cleanup in sync code

chrome/browser/sync/syncable/directory_backing_store.h

 // Copyright (c) 2009 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_SYNCABLE_DIRECTORY_BACKING_STORE_H_
 #define CHROME_BROWSER_SYNC_SYNCABLE_DIRECTORY_BACKING_STORE_H_
 
 #include <set>
 
+#include "base/file_path.h"
 #include "chrome/browser/sync/syncable/dir_open_result.h"
 #include "chrome/browser/sync/syncable/syncable.h"
 
 extern "C" {
 struct sqlite3;
 struct sqlite3_stmt;
 }
 
 namespace syncable {
 
@@ skipping 18 matching lines @@
 //     thread *must* be the thread that owns / is responsible for destroying
 //     the DBS.
 // This way, any thread may open a Directory (which today can be either the
 // AuthWatcherThread or SyncCoreThread) and Load its DBS.  The first time
 // SaveChanges is called a new sqlite3 handle is created, and it will get closed
 // when the DBS is destroyed, which is the reason for the requirement that the
 // thread that "uses" the DBS is the thread that destroys it.
 class DirectoryBackingStore {
  public:
   DirectoryBackingStore(const PathString& dir_name,
-                        const PathString& backing_filepath);
+                        const FilePath& backing_filepath);
 
   virtual ~DirectoryBackingStore();
 
   // Loads and drops all currently persisted meta entries into
   // |entry_bucket|, all currently persisted xattrs in |xattrs_bucket|,
   // and loads appropriate persisted kernel info in |info_bucket|.
   // NOTE: On success (return value of OPENED), the buckets are populated with
   // newly allocated items, meaning ownership is bestowed upon the caller.
   DirOpenResult Load(MetahandlesIndex* entry_bucket,
                      ExtendedAttributes* xattrs_bucket,
@@ skipping 47 matching lines @@
   // The handle to our sqlite on-disk store for initialization and loading, and
   // for saving changes periodically via SaveChanges, respectively.
   // TODO(timsteele): We should only have one handle here.  The reason we need
   // two at the moment is because the DB can be opened by either the AuthWatcher
   // or SyncCore threads, but SaveChanges is always called by the latter.  We
   // need to change initialization so the DB is only accessed from one thread.
   sqlite3* load_dbhandle_;
   sqlite3* save_dbhandle_;
 
   PathString dir_name_;
-  PathString backing_filepath_;
+  FilePath backing_filepath_;
 
   DISALLOW_COPY_AND_ASSIGN(DirectoryBackingStore);
 };
 
 }  // namespace syncable
 
 #endif  // CHROME_BROWSER_SYNC_SYNCABLE_DIRECTORY_BACKING_STORE_H_