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

Unified Diff: chrome/browser/password_manager/encryptor_unittest.cc

Issue 2931020: AutoFill credit cards should be obfuscated on Linux (Closed)
Patch Set: Merge with trunk and Mac changes. Created 10 years, 5 months 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/password_manager/encryptor_linux.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/password_manager/encryptor_unittest.cc
diff --git a/chrome/browser/password_manager/encryptor_unittest.cc b/chrome/browser/password_manager/encryptor_unittest.cc
index 8ca176f6a616df0df82b9bec2657b0986149a6af..4ae41e66dd7edfbec7290994dc6e7db123a3566e 100644
--- a/chrome/browser/password_manager/encryptor_unittest.cc
+++ b/chrome/browser/password_manager/encryptor_unittest.cc
@@ -97,9 +97,6 @@ TEST_F(EncryptorTest, EncryptionDecryption) {
EXPECT_EQ(plaintext, "hel");
}
-// Encryption currently only enabled on Mac and Windows.
-// TODO(dhollowa): http://crbug.com/49115 Implement secure store on Linux.
-#if defined(OS_WIN) || defined(OS_MACOSX)
TEST_F(EncryptorTest, CypherTextDiffers) {
std::string plaintext;
std::string result;
@@ -137,11 +134,10 @@ TEST_F(EncryptorTest, DecryptError) {
ASSERT_TRUE(Encryptor::EncryptString(plaintext, &ciphertext));
EXPECT_NE(plaintext, ciphertext);
ASSERT_LT(4UL, ciphertext.size());
- ciphertext[3] = ciphertext[0] + 1;
+ ciphertext[3] = ciphertext[3] + 1;
EXPECT_FALSE(Encryptor::DecryptString(ciphertext, &result));
EXPECT_NE(plaintext, result);
EXPECT_TRUE(result.empty());
}
-#endif
} // namespace
« no previous file with comments | « chrome/browser/password_manager/encryptor_linux.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698