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

Side by Side Diff: components/os_crypt/os_crypt_mac.mm

Issue 714273004: mac: Expose keychain access frequency to Telemetry. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@mock_keychain_sleep
Patch Set: Respond to offline comments from tonyg. Created 6 years 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/os_crypt/os_crypt.h" 5 #include "components/os_crypt/os_crypt.h"
6 6
7 #include <CommonCrypto/CommonCryptor.h> // for kCCBlockSizeAES128 7 #include <CommonCrypto/CommonCryptor.h> // for kCCBlockSizeAES128
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/debug/leak_annotations.h" 10 #include "base/debug/leak_annotations.h"
11 #include "base/logging.h" 11 #include "base/logging.h"
12 #include "base/memory/scoped_ptr.h" 12 #include "base/memory/scoped_ptr.h"
13 #include "base/perf_util.h"
13 #include "base/strings/utf_string_conversions.h" 14 #include "base/strings/utf_string_conversions.h"
14 #include "base/synchronization/lock.h" 15 #include "base/synchronization/lock.h"
15 #include "components/os_crypt/keychain_password_mac.h" 16 #include "components/os_crypt/keychain_password_mac.h"
16 #include "components/os_crypt/os_crypt_switches.h" 17 #include "components/os_crypt/os_crypt_switches.h"
17 #include "crypto/apple_keychain.h" 18 #include "crypto/apple_keychain.h"
18 #include "crypto/encryptor.h" 19 #include "crypto/encryptor.h"
19 #include "crypto/symmetric_key.h" 20 #include "crypto/symmetric_key.h"
20 21
21 using crypto::AppleKeychain; 22 using crypto::AppleKeychain;
22 23
(...skipping 27 matching lines...) Expand all
50 51
51 if (key_is_cached) 52 if (key_is_cached)
52 return cached_encryption_key; 53 return cached_encryption_key;
53 54
54 static bool mock_keychain_command_line_flag = 55 static bool mock_keychain_command_line_flag =
55 CommandLine::ForCurrentProcess()->HasSwitch( 56 CommandLine::ForCurrentProcess()->HasSwitch(
56 os_crypt::switches::kUseMockKeychain); 57 os_crypt::switches::kUseMockKeychain);
57 58
58 std::string password; 59 std::string password;
59 if (use_mock_keychain || mock_keychain_command_line_flag) { 60 if (use_mock_keychain || mock_keychain_command_line_flag) {
61 base::IncrementKeychainAccessHistogram();
62
60 password = "mock_password"; 63 password = "mock_password";
61 } else { 64 } else {
62 AppleKeychain keychain; 65 AppleKeychain keychain;
63 KeychainPassword encryptor_password(keychain); 66 KeychainPassword encryptor_password(keychain);
64 password = encryptor_password.GetPassword(); 67 password = encryptor_password.GetPassword();
65 } 68 }
66 69
67 // Subsequent code must guarantee that the correct key is cached before 70 // Subsequent code must guarantee that the correct key is cached before
68 // returning. 71 // returning.
69 key_is_cached = true; 72 key_is_cached = true;
(...skipping 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
160 if (!encryptor.Decrypt(raw_ciphertext, plaintext)) 163 if (!encryptor.Decrypt(raw_ciphertext, plaintext))
161 return false; 164 return false;
162 165
163 return true; 166 return true;
164 } 167 }
165 168
166 void OSCrypt::UseMockKeychain(bool use_mock) { 169 void OSCrypt::UseMockKeychain(bool use_mock) {
167 use_mock_keychain = use_mock; 170 use_mock_keychain = use_mock;
168 } 171 }
169 172
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698