summaryrefslogtreecommitdiffstats
path: root/chrome
diff options
context:
space:
mode:
authorchron@google.com <chron@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2010-02-25 23:22:02 +0000
committerchron@google.com <chron@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2010-02-25 23:22:02 +0000
commit6fc1a12eaad5d59282fc27d091abcb2ba95dea11 (patch)
treed5d66a2ab9250d52e6a45cdf608a31d8b13de3a3 /chrome
parentb3c5f8351e6b13b393e60885d863b3a47e0066d6 (diff)
downloadchromium_src-6fc1a12eaad5d59282fc27d091abcb2ba95dea11.zip
chromium_src-6fc1a12eaad5d59282fc27d091abcb2ba95dea11.tar.gz
chromium_src-6fc1a12eaad5d59282fc27d091abcb2ba95dea11.tar.bz2
Temporary fix for commit ID problem. Hex encode IDs so we use ASCII.
Review URL: http://codereview.chromium.org/660108 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40065 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
-rwxr-xr-xchrome/browser/sync/engine/syncapi.cc2
-rwxr-xr-xchrome/browser/sync/engine/syncapi_unittest.cc5
2 files changed, 5 insertions, 2 deletions
diff --git a/chrome/browser/sync/engine/syncapi.cc b/chrome/browser/sync/engine/syncapi.cc
index 43c1d83..7539e2b 100755
--- a/chrome/browser/sync/engine/syncapi.cc
+++ b/chrome/browser/sync/engine/syncapi.cc
@@ -399,7 +399,7 @@ std::string BaseNode::GenerateSyncableHash(
std::string hash_input;
serialized_type.AppendToString(&hash_input);
hash_input.append(client_tag);
- return base::SHA1HashString(hash_input);
+ return HexEncode(hash_input.data(), hash_input.length());
}
int64 BaseNode::GetParentId() const {
diff --git a/chrome/browser/sync/engine/syncapi_unittest.cc b/chrome/browser/sync/engine/syncapi_unittest.cc
index 99597c9..f92d4a3 100755
--- a/chrome/browser/sync/engine/syncapi_unittest.cc
+++ b/chrome/browser/sync/engine/syncapi_unittest.cc
@@ -85,7 +85,9 @@ std::string HashToHex(const std::string& hash) {
}
} // namespace
-TEST_F(SyncApiTest, GenerateSyncableHash) {
+// TODO(chron): Finalize the hash format. Renable test. http://crbug.com/36827
+TEST_F(SyncApiTest, DISABLED_GenerateSyncableHash) {
+/*
EXPECT_EQ("3B2697579984CEB3D2E306E8826B4ABD17DE9002",
HashToHex(BaseNode::GenerateSyncableHash(syncable::BOOKMARKS, "tag1")));
EXPECT_EQ("88D150B511506FE219727D642942446430E42ECE",
@@ -100,6 +102,7 @@ TEST_F(SyncApiTest, GenerateSyncableHash) {
BaseNode::GenerateSyncableHash(syncable::PREFERENCES, "tag2")));
EXPECT_EQ("185896CE8E4D1A18CB94DF8EC827E1CB6F032534",
HashToHex(BaseNode::GenerateSyncableHash(syncable::AUTOFILL, "tag2")));
+*/
}
TEST_F(SyncApiTest, ModelTypesSiloed) {