diff options
author | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-09 21:43:02 +0000 |
---|---|---|
committer | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-09 21:43:02 +0000 |
commit | 74da9bca336135733bb0e2663bc071dda6ed7bbb (patch) | |
tree | e55f700b0163479fa63f81519943ba9bcdc37491 | |
parent | 79c1142fee26cf3ceae688e976fb441339e4b85f (diff) | |
download | chromium_src-74da9bca336135733bb0e2663bc071dda6ed7bbb.zip chromium_src-74da9bca336135733bb0e2663bc071dda6ed7bbb.tar.gz chromium_src-74da9bca336135733bb0e2663bc071dda6ed7bbb.tar.bz2 |
linux: remove a NOTIMPL since we have a bug on it
Also, update some comments to use the new bug number.
BUG=8205,25404
Review URL: http://codereview.chromium.org/483003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34193 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r-- | chrome/browser/password_manager/encryptor_linux.cc | 6 | ||||
-rw-r--r-- | chrome/browser/webdata/web_database.cc | 3 |
2 files changed, 4 insertions, 5 deletions
diff --git a/chrome/browser/password_manager/encryptor_linux.cc b/chrome/browser/password_manager/encryptor_linux.cc index 9c40c37..cc0fa23 100644 --- a/chrome/browser/password_manager/encryptor_linux.cc +++ b/chrome/browser/password_manager/encryptor_linux.cc @@ -25,8 +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. - // http://code.google.com/p/chromium/issues/detail?id=8205 - NOTIMPLEMENTED(); + // http://code.google.com/p/chromium/issues/detail?id=25404 // this does a copy ciphertext->assign(plaintext.data(), plaintext.length()); @@ -36,8 +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. - // http://code.google.com/p/chromium/issues/detail?id=8205 - NOTIMPLEMENTED(); + // http://code.google.com/p/chromium/issues/detail?id=25404 plaintext->assign(ciphertext.data(), ciphertext.length()); return true; diff --git a/chrome/browser/webdata/web_database.cc b/chrome/browser/webdata/web_database.cc index 4d61431..b20ca2c 100644 --- a/chrome/browser/webdata/web_database.cc +++ b/chrome/browser/webdata/web_database.cc @@ -19,7 +19,8 @@ // bottleneck to use the platform methods (e.g. Keychain on the Mac, Gnome // Keyring / KWallet on Linux). That's going to take a massive change in its // API... see: -// http://code.google.com/p/chromium/issues/detail?id=8205 (Linux) +// http://code.google.com/p/chromium/issues/detail?id=25404 (Linux) +// but the (possibly-now-unused) Mac encryptor stub code needs to die too. #include "chrome/browser/password_manager/encryptor.h" using webkit_glue::FormField; |