diff options
| author | georgey@chromium.org <georgey@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-19 23:28:32 +0000 |
|---|---|---|
| committer | georgey@chromium.org <georgey@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-19 23:28:32 +0000 |
| commit | 49580eba25b382f7afa562115e140b42af4be640 (patch) | |
| tree | 60fc488fb589f95b9a8a1067522e7a47684df13f /chrome | |
| parent | 4b3d0b6a088b8d22eae5cd9781cddbe7a0141f98 (diff) | |
| download | chromium_src-49580eba25b382f7afa562115e140b42af4be640.zip chromium_src-49580eba25b382f7afa562115e140b42af4be640.tar.gz chromium_src-49580eba25b382f7afa562115e140b42af4be640.tar.bz2 | |
Removed NOTIMPLEMENTED() from Encryptor::EncryptString() and Encryptor::DecryptString() in
chrome/browser/password_manager/encryptor_mac.mm, so it does exactly the same thing as on Linux: copies unencrypted text.
BUG=25404
TEST=Filling out Autofill for CC on Mac should not assert.
Review URL: http://codereview.chromium.org/1613024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44984 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
| -rw-r--r-- | chrome/browser/password_manager/encryptor_mac.mm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome/browser/password_manager/encryptor_mac.mm b/chrome/browser/password_manager/encryptor_mac.mm index ca133e2..44ed207 100644 --- a/chrome/browser/password_manager/encryptor_mac.mm +++ b/chrome/browser/password_manager/encryptor_mac.mm @@ -25,7 +25,7 @@ bool Encryptor::DecryptString16(const std::string& ciphertext, bool Encryptor::EncryptString(const std::string& plaintext, std::string* ciphertext) { // This doesn't actually encrypt, we need to work on the Encryptor API. - NOTIMPLEMENTED(); + // http://code.google.com/p/chromium/issues/detail?id=25404 // this does a copy ciphertext->assign(plaintext.data(), plaintext.length()); @@ -35,7 +35,7 @@ bool Encryptor::EncryptString(const std::string& plaintext, bool Encryptor::DecryptString(const std::string& ciphertext, std::string* plaintext) { // This doesn't actually decrypt, we need to work on the Encryptor API. - NOTIMPLEMENTED(); + // http://code.google.com/p/chromium/issues/detail?id=25404 plaintext->assign(ciphertext.data(), ciphertext.length()); return true; |