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

Unified Diff: components/password_manager/core/browser/login_database_unittest.cc

Issue 2985373002: Revert of [Password Manager] Send username correction votes (Closed)
Patch Set: Created 3 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
Index: components/password_manager/core/browser/login_database_unittest.cc
diff --git a/components/password_manager/core/browser/login_database_unittest.cc b/components/password_manager/core/browser/login_database_unittest.cc
index 61bc4fad00256ca95d9d883b52a61411d4c499b4..e93ce3f9cba27faaf9dca6322040407169e900ba 100644
--- a/components/password_manager/core/browser/login_database_unittest.cc
+++ b/components/password_manager/core/browser/login_database_unittest.cc
@@ -30,8 +30,6 @@
#include "url/origin.h"
using autofill::PasswordForm;
-using autofill::PossibleUsernamePair;
-using autofill::PossibleUsernamesVector;
using base::ASCIIToUTF16;
using ::testing::Eq;
using ::testing::Pointee;
@@ -132,9 +130,8 @@
} // namespace
// Serialization routines for vectors implemented in login_database.cc.
-base::Pickle SerializePossibleUsernamePairs(const PossibleUsernamesVector& vec);
-PossibleUsernamesVector DeserializePossibleUsernamePairs(
- const base::Pickle& pickle);
+base::Pickle SerializeVector(const std::vector<base::string16>& vec);
+std::vector<base::string16> DeserializeVector(const base::Pickle& pickle);
class LoginDatabaseTest : public testing::Test {
protected:
@@ -966,21 +963,18 @@
TEST_F(LoginDatabaseTest, VectorSerialization) {
// Empty vector.
- PossibleUsernamesVector vec;
- base::Pickle temp = SerializePossibleUsernamePairs(vec);
- PossibleUsernamesVector output = DeserializePossibleUsernamePairs(temp);
+ std::vector<base::string16> vec;
+ base::Pickle temp = SerializeVector(vec);
+ std::vector<base::string16> output = DeserializeVector(temp);
EXPECT_THAT(output, Eq(vec));
// Normal data.
- vec.push_back(
- PossibleUsernamePair(ASCIIToUTF16("first"), ASCIIToUTF16("id1")));
- vec.push_back(
- PossibleUsernamePair(ASCIIToUTF16("second"), ASCIIToUTF16("id2")));
- vec.push_back(
- PossibleUsernamePair(ASCIIToUTF16("third"), ASCIIToUTF16("id3")));
-
- temp = SerializePossibleUsernamePairs(vec);
- output = DeserializePossibleUsernamePairs(temp);
+ vec.push_back(ASCIIToUTF16("first"));
+ vec.push_back(ASCIIToUTF16("second"));
+ vec.push_back(ASCIIToUTF16("third"));
+
+ temp = SerializeVector(vec);
+ output = DeserializeVector(temp);
EXPECT_THAT(output, Eq(vec));
}
@@ -1155,8 +1149,7 @@
form.action = GURL("http://accounts.google.com/login");
form.password_value = ASCIIToUTF16("my_new_password");
form.preferred = false;
- form.other_possible_usernames.push_back(autofill::PossibleUsernamePair(
- ASCIIToUTF16("my_new_username"), ASCIIToUTF16("new_username_id")));
+ form.other_possible_usernames.push_back(ASCIIToUTF16("my_new_username"));
form.times_used = 20;
form.submit_element = ASCIIToUTF16("submit_element");
form.date_synced = base::Time::Now();

Powered by Google App Engine
This is Rietveld 408576698