Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(162)

Unified Diff: chrome/browser/sync/engine/syncapi_internal.cc

Issue 9699057: [Sync] Move 'sync' target to sync/ (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address Tim's comments Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/sync/engine/syncapi_internal.h ('k') | chrome/browser/sync/engine/syncer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/engine/syncapi_internal.cc
diff --git a/chrome/browser/sync/engine/syncapi_internal.cc b/chrome/browser/sync/engine/syncapi_internal.cc
deleted file mode 100644
index ab90c1cb039de5cfb8c95d7814155ab65bfff245..0000000000000000000000000000000000000000
--- a/chrome/browser/sync/engine/syncapi_internal.cc
+++ /dev/null
@@ -1,87 +0,0 @@
-// 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.
-
-#include "chrome/browser/sync/engine/syncapi_internal.h"
-
-#include "base/memory/scoped_ptr.h"
-#include "chrome/browser/sync/util/cryptographer.h"
-#include "sync/protocol/password_specifics.pb.h"
-#include "sync/protocol/sync.pb.h"
-
-using browser_sync::Cryptographer;
-
-namespace sync_api {
-
-sync_pb::PasswordSpecificsData* DecryptPasswordSpecifics(
- const sync_pb::EntitySpecifics& specifics, Cryptographer* crypto) {
- if (!specifics.has_password())
- return NULL;
- const sync_pb::PasswordSpecifics& password_specifics = specifics.password();
- if (!password_specifics.has_encrypted())
- return NULL;
- const sync_pb::EncryptedData& encrypted = password_specifics.encrypted();
- scoped_ptr<sync_pb::PasswordSpecificsData> data(
- new sync_pb::PasswordSpecificsData);
- if (!crypto->Decrypt(encrypted, data.get()))
- return NULL;
- return data.release();
-}
-
-// The list of names which are reserved for use by the server.
-static const char* kForbiddenServerNames[] = { "", ".", ".." };
-
-// When taking a name from the syncapi, append a space if it matches the
-// pattern of a server-illegal name followed by zero or more spaces.
-void SyncAPINameToServerName(const std::string& sync_api_name,
- std::string* out) {
- *out = sync_api_name;
- if (IsNameServerIllegalAfterTrimming(*out))
- out->append(" ");
-}
-
-// Checks whether |name| is a server-illegal name followed by zero or more space
-// characters. The three server-illegal names are the empty string, dot, and
-// dot-dot. Very long names (>255 bytes in UTF-8 Normalization Form C) are
-// also illegal, but are not considered here.
-bool IsNameServerIllegalAfterTrimming(const std::string& name) {
- size_t untrimmed_count = name.find_last_not_of(' ') + 1;
- for (size_t i = 0; i < arraysize(kForbiddenServerNames); ++i) {
- if (name.compare(0, untrimmed_count, kForbiddenServerNames[i]) == 0)
- return true;
- }
- return false;
-}
-
-// Compare the values of two EntitySpecifics, accounting for encryption.
-bool AreSpecificsEqual(const browser_sync::Cryptographer* cryptographer,
- const sync_pb::EntitySpecifics& left,
- const sync_pb::EntitySpecifics& right) {
- // Note that we can't compare encrypted strings directly as they are seeded
- // with a random value.
- std::string left_plaintext, right_plaintext;
- if (left.has_encrypted()) {
- if (!cryptographer->CanDecrypt(left.encrypted())) {
- NOTREACHED() << "Attempting to compare undecryptable data.";
- return false;
- }
- left_plaintext = cryptographer->DecryptToString(left.encrypted());
- } else {
- left_plaintext = left.SerializeAsString();
- }
- if (right.has_encrypted()) {
- if (!cryptographer->CanDecrypt(right.encrypted())) {
- NOTREACHED() << "Attempting to compare undecryptable data.";
- return false;
- }
- right_plaintext = cryptographer->DecryptToString(right.encrypted());
- } else {
- right_plaintext = right.SerializeAsString();
- }
- if (left_plaintext == right_plaintext) {
- return true;
- }
- return false;
-}
-
-} // namespace sync_api
« no previous file with comments | « chrome/browser/sync/engine/syncapi_internal.h ('k') | chrome/browser/sync/engine/syncer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698