String cleanup in sync code

chrome/browser/sync/engine/syncapi.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.
 
 // This file defines the "sync API", an interface to the syncer
 // backend that exposes (1) the core functionality of maintaining a consistent
 // local snapshot of a hierarchical object set; (2) a means to transactionally
 // access and modify those objects; (3) a means to control client/server
 // synchronization tasks, namely: pushing local object modifications to a
 // server, pulling nonlocal object modifications from a server to this client,
@@ skipping 21 matching lines @@
 // Consequently, it does not support looking up an object by title
 // and parent, since such a lookup is not uniquely determined.  Lastly,
 // unlike a filesystem model, objects in the Sync API model have a strict
 // ordering within a parent; the position is manipulable by callers, and
 // children of a node can be enumerated in the order of their position.
 
 #ifndef CHROME_BROWSER_SYNC_ENGINE_SYNCAPI_H_
 #define CHROME_BROWSER_SYNC_ENGINE_SYNCAPI_H_
 
 #include "base/basictypes.h"
+#include "base/file_path.h"
 #include "build/build_config.h"
 
 #if defined(OS_WIN)
 typedef wchar_t sync_char16;
 #else
 typedef uint16 sync_char16;
 #endif
 
 // The MSVC compiler for Windows requires that any classes exported by, or
 // imported from, a dynamic library be marked with an appropriate
@@ skipping 464 matching lines @@
   // code live on separate threads that run simultaneously, we just dedicate
   // one to each component. Long term we may want to reconsider the HttpBridge
   // API to take all the params in one chunk in a threadsafe manner.. which is
   // still suboptimal as there will be high contention between the two threads
   // on startup; so maybe what we have now is the best solution- it does mirror
   // the CURL implementation as each thread creates their own internet handle.
   // Investigate.
   // |model_safe_worker| ownership is given to the SyncManager.
   // |user_agent| is a 7-bit ASCII string suitable for use as the User-Agent
   // HTTP header. Used internally when collecting stats to classify clients.
-  bool Init(const sync_char16* database_location,
+  bool Init(const FilePath& database_location,
             const char* sync_server_and_path,
             int sync_server_port,
             const char* gaia_service_id,
             const char* gaia_source,
             bool use_ssl,
             HttpPostProviderFactory* post_factory,
             HttpPostProviderFactory* auth_post_factory,
             ModelSafeWorkerInterface* model_safe_worker,
             bool attempt_last_user_authentication,
             const char* user_agent);
@@ skipping 146 matching lines @@
   // Subclasses should implement to invoke DoWork on |visitor| once on a thread
   // appropriate for data model modifications.
   // While it doesn't hurt, the impl does not need to be re-entrant (for now).
   // Note: |visitor| is owned by caller.
   virtual void CallDoWorkFromModelSafeThreadAndWait(Visitor* visitor) = 0;
 };
 
 }  // namespace sync_api
 
 #endif  // CHROME_BROWSER_SYNC_ENGINE_SYNCAPI_H_