summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorevan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-12-09 21:43:02 +0000
committerevan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-12-09 21:43:02 +0000
commit74da9bca336135733bb0e2663bc071dda6ed7bbb (patch)
treee55f700b0163479fa63f81519943ba9bcdc37491
parent79c1142fee26cf3ceae688e976fb441339e4b85f (diff)
downloadchromium_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.cc6
-rw-r--r--chrome/browser/webdata/web_database.cc3
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;