Refactor syncapi.h

chrome/browser/sync/engine/syncapi_internal.h

Index: chrome/browser/sync/engine/syncapi_internal.h
diff --git a/chrome/browser/sync/engine/syncapi_internal.h b/chrome/browser/sync/engine/syncapi_internal.h
new file mode 100644
index 0000000000000000000000000000000000000000..277acc991e2a7b2f281c79f7fbfdebfc86a6258f
--- /dev/null
+++ b/chrome/browser/sync/engine/syncapi_internal.h
@@ -0,0 +1,34 @@
+// 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_ENGINE_SYNCAPI_INTERNAL_H_
+#define CHROME_BROWSER_SYNC_ENGINE_SYNCAPI_INTERNAL_H_
+
+// The functions defined are shared among some of the classes that implement
+// the internal sync_api.  They are not to be used by clients of the API.

[tim (not reviewing), 2011/08/15 22:53:55]

Hmm. Ideally these would stay under /engine/ then, so we can enforce proper DEPS
rules.  Although "internal" is about to get overloaded :\

[rlarocque, 2011/08/16 18:12:46]

Done.

+
+#include <string>
+
+namespace browser_sync {
+class Cryptographer;
+}
+
+namespace sync_pb {
+class EntitySpecifics;
+class PasswordSpecificsData;
+}
+
+namespace sync_api {
+sync_pb::PasswordSpecificsData* DecryptPasswordSpecifics(
+    const sync_pb::EntitySpecifics& specifics,
+    browser_sync::Cryptographer* crypto);
+
+bool IsNameServerIllegalAfterTrimming(const std::string& name);
+
+bool AreSpecificsEqual(const browser_sync::Cryptographer* cryptographer,
+                         const sync_pb::EntitySpecifics& left,
+                         const sync_pb::EntitySpecifics& right);
+}
+
+#endif  // CHROME_BROWSER_SYNC_ENGINE_SYNCAPI_INTERNAL_H_