Move *keychain_mac* files to crypto/

crypto/keychain_mac.h

-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// 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.
 
-#ifndef CHROME_BROWSER_KEYCHAIN_MAC_H_
-#define CHROME_BROWSER_KEYCHAIN_MAC_H_
+#ifndef CRYPTO_KEYCHAIN_MAC_H_
+#define CRYPTO_KEYCHAIN_MAC_H_
 #pragma once
 
 #include <Security/Security.h>
 
 #include "base/basictypes.h"
 
+namespace crypto {
+
 // Wraps the KeychainServices API in a very thin layer, to allow it to be
 // mocked out for testing.
 
 // See Keychain Services documentation for function documentation, as these call
 // through directly to their Keychain Services equivalents (Foo ->
 // SecKeychainFoo). The only exception is Free, which should be used for
 // anything returned from this class that would normally be freed with
 // CFRelease (to aid in testing).
 class MacKeychain {
  public:
   MacKeychain() {}
   virtual ~MacKeychain() {}
 
   virtual OSStatus ItemCopyAttributesAndData(
       SecKeychainItemRef itemRef, SecKeychainAttributeInfo *info,
       SecItemClass *itemClass, SecKeychainAttributeList **attrList,
       UInt32 *length, void **outData) const;

[Ryan Sleevi, 2012/03/16 20:58:37]

Can you update these type names to have the * with the typename, since it's part
of the type, as per
http://www.chromium.org/developers/coding-style#TOC-Code-formatting

eg:

SecKeychainAttributeInfo *info -> SecKeychainAttributeInfo* info

Also, line breaks/whitespace since the function declaration doesn't all fit on
one line.

[akalin, 2012/03/17 07:19:44]

I assume you mean the recommended one-arg-per-line style?  Done.

 
   virtual OSStatus ItemModifyAttributesAndData(
       SecKeychainItemRef itemRef, const SecKeychainAttributeList *attrList,
       UInt32 length, const void *data) const;
 
   virtual OSStatus ItemFreeAttributesAndData(SecKeychainAttributeList *attrList,
                                              void *data) const;
 
   virtual OSStatus ItemDelete(SecKeychainItemRef itemRef) const;
 
@@ skipping 40 matching lines @@
                                       const void *passwordData,
                                       SecKeychainItemRef *itemRef) const;
 
   // Calls CFRelease on the given ref, after checking that |ref| is non-NULL.
   virtual void Free(CFTypeRef ref) const;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(MacKeychain);
 };
 
-#endif  // CHROME_BROWSER_KEYCHAIN_MAC_H_
+}  // namespace crypto
+
+#endif  // CRYPTO_KEYCHAIN_MAC_H_