Move crypto_helpers from sync to base

crypto/md5_calculator.h

Index: crypto/md5_calculator.h
diff --git a/chrome/browser/sync/util/crypto_helpers.h b/crypto/md5_calculator.h
similarity index 69%
rename from chrome/browser/sync/util/crypto_helpers.h
rename to crypto/md5_calculator.h
index 34b36f477b89544faefc034d3e952691b56fedb7..da3d5a770ea5bb47d53166fdb65a66b9cb449954 100644
--- a/chrome/browser/sync/util/crypto_helpers.h
+++ b/crypto/md5_calculator.h
@@ -1,19 +1,21 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// 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_UTIL_CRYPTO_HELPERS_H_
-#define CHROME_BROWSER_SYNC_UTIL_CRYPTO_HELPERS_H_
+#ifndef CRYPTO_MD5_CALCULATOR_H_
+#define CRYPTO_MD5_CALCULATOR_H_
 #pragma once
 
 #include <string>
 #include <vector>
 
-// An object to handle calculation of MD5 sums.
 #include "base/basictypes.h"
 #include "base/md5.h"
 #include "base/port.h"
+#include "base/string_number_conversions.h"
+#include "base/string_util.h"
 
+// An object to handle calculation of MD5 sums.
 class MD5Calculator {

[agl, 2011/04/26 13:59:37]

I'm afraid that this wrapper object is too thin to be worthwhile. The (single?)
user should simply be updated to use the underlying base/md5.h because this adds
almost nothing.

  protected:
   MD5Context context_;
@@ -33,7 +35,4 @@ class MD5Calculator {
   DISALLOW_COPY_AND_ASSIGN(MD5Calculator);
 };
 
-void GetRandomBytes(char* output, int output_length);
-std::string Generate128BitRandomHexString();
-
-#endif  // CHROME_BROWSER_SYNC_UTIL_CRYPTO_HELPERS_H_
+#endif  // CRYPTO_MD5_CALCULATOR_H_