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

Side by Side Diff: chrome/browser/ui/passwords/manage_passwords_state_unittest.cc

Issue 1550053002: Convert Pass()→std::move() in //chrome/browser/ui (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 12 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/passwords/manage_passwords_state.h" 5 #include "chrome/browser/ui/passwords/manage_passwords_state.h"
6 6
7 #include <utility> 7 #include <utility>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/strings/utf_string_conversions.h" 10 #include "base/strings/utf_string_conversions.h"
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 79
80 scoped_ptr<password_manager::PasswordFormManager> 80 scoped_ptr<password_manager::PasswordFormManager>
81 ManagePasswordsStateTest::CreateFormManager() { 81 ManagePasswordsStateTest::CreateFormManager() {
82 scoped_ptr<password_manager::PasswordFormManager> test_form_manager( 82 scoped_ptr<password_manager::PasswordFormManager> test_form_manager(
83 new password_manager::PasswordFormManager( 83 new password_manager::PasswordFormManager(
84 &password_manager_, &stub_client_, driver_.AsWeakPtr(), 84 &password_manager_, &stub_client_, driver_.AsWeakPtr(),
85 test_local_form(), false)); 85 test_local_form(), false));
86 test_form_manager->SimulateFetchMatchingLoginsFromPasswordStore(); 86 test_form_manager->SimulateFetchMatchingLoginsFromPasswordStore();
87 ScopedVector<autofill::PasswordForm> stored_forms; 87 ScopedVector<autofill::PasswordForm> stored_forms;
88 stored_forms.push_back(new autofill::PasswordForm(test_local_form())); 88 stored_forms.push_back(new autofill::PasswordForm(test_local_form()));
89 test_form_manager->OnGetPasswordStoreResults(stored_forms.Pass()); 89 test_form_manager->OnGetPasswordStoreResults(std::move(stored_forms));
90 EXPECT_EQ(1u, test_form_manager->best_matches().size()); 90 EXPECT_EQ(1u, test_form_manager->best_matches().size());
91 EXPECT_EQ(test_local_form(), 91 EXPECT_EQ(test_local_form(),
92 *test_form_manager->best_matches().begin()->second); 92 *test_form_manager->best_matches().begin()->second);
93 return test_form_manager.Pass(); 93 return test_form_manager;
94 } 94 }
95 95
96 void ManagePasswordsStateTest::TestNoisyUpdates() { 96 void ManagePasswordsStateTest::TestNoisyUpdates() {
97 const std::vector<const autofill::PasswordForm*> forms = 97 const std::vector<const autofill::PasswordForm*> forms =
98 passwords_data_.GetCurrentForms(); 98 passwords_data_.GetCurrentForms();
99 const std::vector<const autofill::PasswordForm*> federated_forms = 99 const std::vector<const autofill::PasswordForm*> federated_forms =
100 passwords_data_.federated_credentials_forms(); 100 passwords_data_.federated_credentials_forms();
101 const password_manager::ui::State state = passwords_data_.state(); 101 const password_manager::ui::State state = passwords_data_.state();
102 const GURL origin = passwords_data_.origin(); 102 const GURL origin = passwords_data_.origin();
103 103
(...skipping 117 matching lines...) Expand 10 before | Expand all | Expand 10 after
221 221
222 TestNoisyUpdates(); 222 TestNoisyUpdates();
223 } 223 }
224 224
225 TEST_F(ManagePasswordsStateTest, PasswordSubmitted) { 225 TEST_F(ManagePasswordsStateTest, PasswordSubmitted) {
226 scoped_ptr<password_manager::PasswordFormManager> test_form_manager( 226 scoped_ptr<password_manager::PasswordFormManager> test_form_manager(
227 CreateFormManager()); 227 CreateFormManager());
228 test_form_manager->ProvisionallySave( 228 test_form_manager->ProvisionallySave(
229 test_submitted_form(), 229 test_submitted_form(),
230 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES); 230 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES);
231 passwords_data().OnPendingPassword(test_form_manager.Pass()); 231 passwords_data().OnPendingPassword(std::move(test_form_manager));
232 232
233 EXPECT_THAT(passwords_data().GetCurrentForms(), 233 EXPECT_THAT(passwords_data().GetCurrentForms(),
234 ElementsAre(Pointee(test_local_form()))); 234 ElementsAre(Pointee(test_local_form())));
235 EXPECT_THAT(passwords_data().federated_credentials_forms(), IsEmpty()); 235 EXPECT_THAT(passwords_data().federated_credentials_forms(), IsEmpty());
236 EXPECT_EQ(password_manager::ui::PENDING_PASSWORD_STATE, 236 EXPECT_EQ(password_manager::ui::PENDING_PASSWORD_STATE,
237 passwords_data().state()); 237 passwords_data().state());
238 EXPECT_EQ(test_submitted_form().origin, passwords_data().origin()); 238 EXPECT_EQ(test_submitted_form().origin, passwords_data().origin());
239 ASSERT_TRUE(passwords_data().form_manager()); 239 ASSERT_TRUE(passwords_data().form_manager());
240 EXPECT_EQ(test_submitted_form(), 240 EXPECT_EQ(test_submitted_form(),
241 passwords_data().form_manager()->pending_credentials()); 241 passwords_data().form_manager()->pending_credentials());
242 TestAllUpdates(); 242 TestAllUpdates();
243 } 243 }
244 244
245 TEST_F(ManagePasswordsStateTest, PasswordSaved) { 245 TEST_F(ManagePasswordsStateTest, PasswordSaved) {
246 scoped_ptr<password_manager::PasswordFormManager> test_form_manager( 246 scoped_ptr<password_manager::PasswordFormManager> test_form_manager(
247 CreateFormManager()); 247 CreateFormManager());
248 test_form_manager->ProvisionallySave( 248 test_form_manager->ProvisionallySave(
249 test_submitted_form(), 249 test_submitted_form(),
250 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES); 250 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES);
251 passwords_data().OnPendingPassword(test_form_manager.Pass()); 251 passwords_data().OnPendingPassword(std::move(test_form_manager));
252 EXPECT_EQ(password_manager::ui::PENDING_PASSWORD_STATE, 252 EXPECT_EQ(password_manager::ui::PENDING_PASSWORD_STATE,
253 passwords_data().state()); 253 passwords_data().state());
254 254
255 passwords_data().TransitionToState(password_manager::ui::MANAGE_STATE); 255 passwords_data().TransitionToState(password_manager::ui::MANAGE_STATE);
256 EXPECT_THAT(passwords_data().GetCurrentForms(), 256 EXPECT_THAT(passwords_data().GetCurrentForms(),
257 ElementsAre(Pointee(test_local_form()))); 257 ElementsAre(Pointee(test_local_form())));
258 EXPECT_THAT(passwords_data().federated_credentials_forms(), IsEmpty()); 258 EXPECT_THAT(passwords_data().federated_credentials_forms(), IsEmpty());
259 EXPECT_EQ(password_manager::ui::MANAGE_STATE, 259 EXPECT_EQ(password_manager::ui::MANAGE_STATE,
260 passwords_data().state()); 260 passwords_data().state());
261 EXPECT_EQ(test_submitted_form().origin, passwords_data().origin()); 261 EXPECT_EQ(test_submitted_form().origin, passwords_data().origin());
262 TestAllUpdates(); 262 TestAllUpdates();
263 } 263 }
264 264
265 TEST_F(ManagePasswordsStateTest, OnRequestCredentials) { 265 TEST_F(ManagePasswordsStateTest, OnRequestCredentials) {
266 ScopedVector<autofill::PasswordForm> local_credentials; 266 ScopedVector<autofill::PasswordForm> local_credentials;
267 local_credentials.push_back(new autofill::PasswordForm(test_local_form())); 267 local_credentials.push_back(new autofill::PasswordForm(test_local_form()));
268 ScopedVector<autofill::PasswordForm> federated_credentials; 268 ScopedVector<autofill::PasswordForm> federated_credentials;
269 federated_credentials.push_back( 269 federated_credentials.push_back(
270 new autofill::PasswordForm(test_federated_form())); 270 new autofill::PasswordForm(test_federated_form()));
271 const GURL origin = test_local_form().origin; 271 const GURL origin = test_local_form().origin;
272 passwords_data().OnRequestCredentials(local_credentials.Pass(), 272 passwords_data().OnRequestCredentials(
273 federated_credentials.Pass(), origin); 273 std::move(local_credentials), std::move(federated_credentials), origin);
274 passwords_data().set_credentials_callback(base::Bind( 274 passwords_data().set_credentials_callback(base::Bind(
275 &ManagePasswordsStateTest::CredentialCallback, base::Unretained(this))); 275 &ManagePasswordsStateTest::CredentialCallback, base::Unretained(this)));
276 EXPECT_THAT(passwords_data().GetCurrentForms(), 276 EXPECT_THAT(passwords_data().GetCurrentForms(),
277 ElementsAre(Pointee(test_local_form()))); 277 ElementsAre(Pointee(test_local_form())));
278 EXPECT_THAT(passwords_data().federated_credentials_forms(), 278 EXPECT_THAT(passwords_data().federated_credentials_forms(),
279 ElementsAre(Pointee(test_federated_form()))); 279 ElementsAre(Pointee(test_federated_form())));
280 EXPECT_EQ(password_manager::ui::CREDENTIAL_REQUEST_STATE, 280 EXPECT_EQ(password_manager::ui::CREDENTIAL_REQUEST_STATE,
281 passwords_data().state()); 281 passwords_data().state());
282 EXPECT_EQ(origin, passwords_data().origin()); 282 EXPECT_EQ(origin, passwords_data().origin());
283 TestAllUpdates(); 283 TestAllUpdates();
(...skipping 11 matching lines...) Expand all
295 ElementsAre(Pointee(test_local_form()))); 295 ElementsAre(Pointee(test_local_form())));
296 EXPECT_THAT(passwords_data().federated_credentials_forms(), IsEmpty()); 296 EXPECT_THAT(passwords_data().federated_credentials_forms(), IsEmpty());
297 EXPECT_EQ(password_manager::ui::MANAGE_STATE, passwords_data().state()); 297 EXPECT_EQ(password_manager::ui::MANAGE_STATE, passwords_data().state());
298 EXPECT_EQ(origin, passwords_data().origin()); 298 EXPECT_EQ(origin, passwords_data().origin());
299 TestAllUpdates(); 299 TestAllUpdates();
300 } 300 }
301 301
302 TEST_F(ManagePasswordsStateTest, AutoSignin) { 302 TEST_F(ManagePasswordsStateTest, AutoSignin) {
303 ScopedVector<autofill::PasswordForm> local_credentials; 303 ScopedVector<autofill::PasswordForm> local_credentials;
304 local_credentials.push_back(new autofill::PasswordForm(test_local_form())); 304 local_credentials.push_back(new autofill::PasswordForm(test_local_form()));
305 passwords_data().OnAutoSignin(local_credentials.Pass()); 305 passwords_data().OnAutoSignin(std::move(local_credentials));
306 EXPECT_THAT(passwords_data().GetCurrentForms(), 306 EXPECT_THAT(passwords_data().GetCurrentForms(),
307 ElementsAre(Pointee(test_local_form()))); 307 ElementsAre(Pointee(test_local_form())));
308 EXPECT_THAT(passwords_data().federated_credentials_forms(), IsEmpty()); 308 EXPECT_THAT(passwords_data().federated_credentials_forms(), IsEmpty());
309 EXPECT_EQ(password_manager::ui::AUTO_SIGNIN_STATE, passwords_data().state()); 309 EXPECT_EQ(password_manager::ui::AUTO_SIGNIN_STATE, passwords_data().state());
310 EXPECT_EQ(test_local_form().origin, passwords_data().origin()); 310 EXPECT_EQ(test_local_form().origin, passwords_data().origin());
311 TestAllUpdates(); 311 TestAllUpdates();
312 312
313 passwords_data().TransitionToState(password_manager::ui::MANAGE_STATE); 313 passwords_data().TransitionToState(password_manager::ui::MANAGE_STATE);
314 EXPECT_THAT(passwords_data().GetCurrentForms(), 314 EXPECT_THAT(passwords_data().GetCurrentForms(),
315 ElementsAre(Pointee(test_local_form()))); 315 ElementsAre(Pointee(test_local_form())));
316 EXPECT_THAT(passwords_data().federated_credentials_forms(), IsEmpty()); 316 EXPECT_THAT(passwords_data().federated_credentials_forms(), IsEmpty());
317 EXPECT_EQ(password_manager::ui::MANAGE_STATE, passwords_data().state()); 317 EXPECT_EQ(password_manager::ui::MANAGE_STATE, passwords_data().state());
318 EXPECT_EQ(test_local_form().origin, passwords_data().origin()); 318 EXPECT_EQ(test_local_form().origin, passwords_data().origin());
319 TestAllUpdates(); 319 TestAllUpdates();
320 } 320 }
321 321
322 TEST_F(ManagePasswordsStateTest, AutomaticPasswordSave) { 322 TEST_F(ManagePasswordsStateTest, AutomaticPasswordSave) {
323 scoped_ptr<password_manager::PasswordFormManager> test_form_manager( 323 scoped_ptr<password_manager::PasswordFormManager> test_form_manager(
324 CreateFormManager()); 324 CreateFormManager());
325 test_form_manager->ProvisionallySave( 325 test_form_manager->ProvisionallySave(
326 test_submitted_form(), 326 test_submitted_form(),
327 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES); 327 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES);
328 328
329 passwords_data().OnAutomaticPasswordSave(test_form_manager.Pass()); 329 passwords_data().OnAutomaticPasswordSave(std::move(test_form_manager));
330 EXPECT_EQ(password_manager::ui::CONFIRMATION_STATE, passwords_data().state()); 330 EXPECT_EQ(password_manager::ui::CONFIRMATION_STATE, passwords_data().state());
331 EXPECT_EQ(test_submitted_form().origin, passwords_data().origin()); 331 EXPECT_EQ(test_submitted_form().origin, passwords_data().origin());
332 ASSERT_TRUE(passwords_data().form_manager()); 332 ASSERT_TRUE(passwords_data().form_manager());
333 EXPECT_EQ(test_submitted_form(), 333 EXPECT_EQ(test_submitted_form(),
334 passwords_data().form_manager()->pending_credentials()); 334 passwords_data().form_manager()->pending_credentials());
335 TestAllUpdates(); 335 TestAllUpdates();
336 336
337 passwords_data().TransitionToState(password_manager::ui::MANAGE_STATE); 337 passwords_data().TransitionToState(password_manager::ui::MANAGE_STATE);
338 EXPECT_THAT(passwords_data().GetCurrentForms(), 338 EXPECT_THAT(passwords_data().GetCurrentForms(),
339 UnorderedElementsAre(Pointee(test_local_form()), 339 UnorderedElementsAre(Pointee(test_local_form()),
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
380 EXPECT_EQ(GURL::EmptyGURL(), passwords_data().origin()); 380 EXPECT_EQ(GURL::EmptyGURL(), passwords_data().origin());
381 EXPECT_FALSE(passwords_data().form_manager()); 381 EXPECT_FALSE(passwords_data().form_manager());
382 } 382 }
383 383
384 TEST_F(ManagePasswordsStateTest, OnInactive) { 384 TEST_F(ManagePasswordsStateTest, OnInactive) {
385 scoped_ptr<password_manager::PasswordFormManager> test_form_manager( 385 scoped_ptr<password_manager::PasswordFormManager> test_form_manager(
386 CreateFormManager()); 386 CreateFormManager());
387 test_form_manager->ProvisionallySave( 387 test_form_manager->ProvisionallySave(
388 test_submitted_form(), 388 test_submitted_form(),
389 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES); 389 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES);
390 passwords_data().OnPendingPassword(test_form_manager.Pass()); 390 passwords_data().OnPendingPassword(std::move(test_form_manager));
391 EXPECT_EQ(password_manager::ui::PENDING_PASSWORD_STATE, 391 EXPECT_EQ(password_manager::ui::PENDING_PASSWORD_STATE,
392 passwords_data().state()); 392 passwords_data().state());
393 passwords_data().OnInactive(); 393 passwords_data().OnInactive();
394 EXPECT_THAT(passwords_data().GetCurrentForms(), IsEmpty()); 394 EXPECT_THAT(passwords_data().GetCurrentForms(), IsEmpty());
395 EXPECT_THAT(passwords_data().federated_credentials_forms(), IsEmpty()); 395 EXPECT_THAT(passwords_data().federated_credentials_forms(), IsEmpty());
396 EXPECT_EQ(password_manager::ui::INACTIVE_STATE, passwords_data().state()); 396 EXPECT_EQ(password_manager::ui::INACTIVE_STATE, passwords_data().state());
397 EXPECT_EQ(GURL::EmptyGURL(), passwords_data().origin()); 397 EXPECT_EQ(GURL::EmptyGURL(), passwords_data().origin());
398 EXPECT_FALSE(passwords_data().form_manager()); 398 EXPECT_FALSE(passwords_data().form_manager());
399 TestNoisyUpdates(); 399 TestNoisyUpdates();
400 } 400 }
401 401
402 TEST_F(ManagePasswordsStateTest, PendingPasswordAddBlacklisted) { 402 TEST_F(ManagePasswordsStateTest, PendingPasswordAddBlacklisted) {
403 scoped_ptr<password_manager::PasswordFormManager> test_form_manager( 403 scoped_ptr<password_manager::PasswordFormManager> test_form_manager(
404 CreateFormManager()); 404 CreateFormManager());
405 test_form_manager->ProvisionallySave( 405 test_form_manager->ProvisionallySave(
406 test_submitted_form(), 406 test_submitted_form(),
407 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES); 407 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES);
408 passwords_data().OnPendingPassword(test_form_manager.Pass()); 408 passwords_data().OnPendingPassword(std::move(test_form_manager));
409 EXPECT_EQ(password_manager::ui::PENDING_PASSWORD_STATE, 409 EXPECT_EQ(password_manager::ui::PENDING_PASSWORD_STATE,
410 passwords_data().state()); 410 passwords_data().state());
411 411
412 TestBlacklistedUpdates(); 412 TestBlacklistedUpdates();
413 } 413 }
414 414
415 TEST_F(ManagePasswordsStateTest, RequestCredentialsAddBlacklisted) { 415 TEST_F(ManagePasswordsStateTest, RequestCredentialsAddBlacklisted) {
416 ScopedVector<autofill::PasswordForm> local_credentials; 416 ScopedVector<autofill::PasswordForm> local_credentials;
417 local_credentials.push_back(new autofill::PasswordForm(test_local_form())); 417 local_credentials.push_back(new autofill::PasswordForm(test_local_form()));
418 ScopedVector<autofill::PasswordForm> federated_credentials; 418 ScopedVector<autofill::PasswordForm> federated_credentials;
419 federated_credentials.push_back( 419 federated_credentials.push_back(
420 new autofill::PasswordForm(test_federated_form())); 420 new autofill::PasswordForm(test_federated_form()));
421 const GURL origin = test_local_form().origin; 421 const GURL origin = test_local_form().origin;
422 passwords_data().OnRequestCredentials(local_credentials.Pass(), 422 passwords_data().OnRequestCredentials(
423 federated_credentials.Pass(), origin); 423 std::move(local_credentials), std::move(federated_credentials), origin);
424 passwords_data().set_credentials_callback(base::Bind( 424 passwords_data().set_credentials_callback(base::Bind(
425 &ManagePasswordsStateTest::CredentialCallback, base::Unretained(this))); 425 &ManagePasswordsStateTest::CredentialCallback, base::Unretained(this)));
426 EXPECT_EQ(password_manager::ui::CREDENTIAL_REQUEST_STATE, 426 EXPECT_EQ(password_manager::ui::CREDENTIAL_REQUEST_STATE,
427 passwords_data().state()); 427 passwords_data().state());
428 428
429 TestBlacklistedUpdates(); 429 TestBlacklistedUpdates();
430 } 430 }
431 431
432 TEST_F(ManagePasswordsStateTest, AutoSigninAddBlacklisted) { 432 TEST_F(ManagePasswordsStateTest, AutoSigninAddBlacklisted) {
433 ScopedVector<autofill::PasswordForm> local_credentials; 433 ScopedVector<autofill::PasswordForm> local_credentials;
434 local_credentials.push_back(new autofill::PasswordForm(test_local_form())); 434 local_credentials.push_back(new autofill::PasswordForm(test_local_form()));
435 passwords_data().OnAutoSignin(local_credentials.Pass()); 435 passwords_data().OnAutoSignin(std::move(local_credentials));
436 EXPECT_EQ(password_manager::ui::AUTO_SIGNIN_STATE, passwords_data().state()); 436 EXPECT_EQ(password_manager::ui::AUTO_SIGNIN_STATE, passwords_data().state());
437 437
438 TestBlacklistedUpdates(); 438 TestBlacklistedUpdates();
439 } 439 }
440 440
441 TEST_F(ManagePasswordsStateTest, AutomaticPasswordSaveAddBlacklisted) { 441 TEST_F(ManagePasswordsStateTest, AutomaticPasswordSaveAddBlacklisted) {
442 scoped_ptr<password_manager::PasswordFormManager> test_form_manager( 442 scoped_ptr<password_manager::PasswordFormManager> test_form_manager(
443 CreateFormManager()); 443 CreateFormManager());
444 test_form_manager->ProvisionallySave( 444 test_form_manager->ProvisionallySave(
445 test_submitted_form(), 445 test_submitted_form(),
446 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES); 446 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES);
447 passwords_data().OnAutomaticPasswordSave(test_form_manager.Pass()); 447 passwords_data().OnAutomaticPasswordSave(std::move(test_form_manager));
448 EXPECT_EQ(password_manager::ui::CONFIRMATION_STATE, passwords_data().state()); 448 EXPECT_EQ(password_manager::ui::CONFIRMATION_STATE, passwords_data().state());
449 449
450 TestBlacklistedUpdates(); 450 TestBlacklistedUpdates();
451 } 451 }
452 452
453 TEST_F(ManagePasswordsStateTest, BackgroundAutofilledAddBlacklisted) { 453 TEST_F(ManagePasswordsStateTest, BackgroundAutofilledAddBlacklisted) {
454 autofill::PasswordFormMap password_form_map; 454 autofill::PasswordFormMap password_form_map;
455 password_form_map.insert(std::make_pair( 455 password_form_map.insert(std::make_pair(
456 test_local_form().username_value, 456 test_local_form().username_value,
457 make_scoped_ptr(new autofill::PasswordForm(test_local_form())))); 457 make_scoped_ptr(new autofill::PasswordForm(test_local_form()))));
458 passwords_data().OnPasswordAutofilled( 458 passwords_data().OnPasswordAutofilled(
459 password_form_map, password_form_map.begin()->second->origin); 459 password_form_map, password_form_map.begin()->second->origin);
460 EXPECT_EQ(password_manager::ui::MANAGE_STATE, passwords_data().state()); 460 EXPECT_EQ(password_manager::ui::MANAGE_STATE, passwords_data().state());
461 461
462 TestBlacklistedUpdates(); 462 TestBlacklistedUpdates();
463 } 463 }
464 464
465 TEST_F(ManagePasswordsStateTest, PasswordUpdateAddBlacklisted) { 465 TEST_F(ManagePasswordsStateTest, PasswordUpdateAddBlacklisted) {
466 scoped_ptr<password_manager::PasswordFormManager> test_form_manager( 466 scoped_ptr<password_manager::PasswordFormManager> test_form_manager(
467 CreateFormManager()); 467 CreateFormManager());
468 test_form_manager->ProvisionallySave( 468 test_form_manager->ProvisionallySave(
469 test_submitted_form(), 469 test_submitted_form(),
470 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES); 470 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES);
471 passwords_data().OnUpdatePassword(test_form_manager.Pass()); 471 passwords_data().OnUpdatePassword(std::move(test_form_manager));
472 EXPECT_EQ(password_manager::ui::PENDING_PASSWORD_UPDATE_STATE, 472 EXPECT_EQ(password_manager::ui::PENDING_PASSWORD_UPDATE_STATE,
473 passwords_data().state()); 473 passwords_data().state());
474 474
475 TestBlacklistedUpdates(); 475 TestBlacklistedUpdates();
476 } 476 }
477 477
478 TEST_F(ManagePasswordsStateTest, PasswordUpdateSubmitted) { 478 TEST_F(ManagePasswordsStateTest, PasswordUpdateSubmitted) {
479 scoped_ptr<password_manager::PasswordFormManager> test_form_manager( 479 scoped_ptr<password_manager::PasswordFormManager> test_form_manager(
480 CreateFormManager()); 480 CreateFormManager());
481 test_form_manager->ProvisionallySave( 481 test_form_manager->ProvisionallySave(
482 test_submitted_form(), 482 test_submitted_form(),
483 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES); 483 password_manager::PasswordFormManager::IGNORE_OTHER_POSSIBLE_USERNAMES);
484 passwords_data().OnUpdatePassword(test_form_manager.Pass()); 484 passwords_data().OnUpdatePassword(std::move(test_form_manager));
485 485
486 EXPECT_THAT(passwords_data().GetCurrentForms(), 486 EXPECT_THAT(passwords_data().GetCurrentForms(),
487 ElementsAre(Pointee(test_local_form()))); 487 ElementsAre(Pointee(test_local_form())));
488 EXPECT_THAT(passwords_data().federated_credentials_forms(), IsEmpty()); 488 EXPECT_THAT(passwords_data().federated_credentials_forms(), IsEmpty());
489 EXPECT_EQ(password_manager::ui::PENDING_PASSWORD_UPDATE_STATE, 489 EXPECT_EQ(password_manager::ui::PENDING_PASSWORD_UPDATE_STATE,
490 passwords_data().state()); 490 passwords_data().state());
491 EXPECT_EQ(test_submitted_form().origin, passwords_data().origin()); 491 EXPECT_EQ(test_submitted_form().origin, passwords_data().origin());
492 ASSERT_TRUE(passwords_data().form_manager()); 492 ASSERT_TRUE(passwords_data().form_manager());
493 EXPECT_EQ(test_submitted_form(), 493 EXPECT_EQ(test_submitted_form(),
494 passwords_data().form_manager()->pending_credentials()); 494 passwords_data().form_manager()->pending_credentials());
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
550 passwords_data().set_credentials_callback(base::Bind( 550 passwords_data().set_credentials_callback(base::Bind(
551 &ManagePasswordsStateTest::CredentialCallback, base::Unretained(this))); 551 &ManagePasswordsStateTest::CredentialCallback, base::Unretained(this)));
552 password_manager::CredentialInfo credential_info( 552 password_manager::CredentialInfo credential_info(
553 form, password_manager::CredentialType::CREDENTIAL_TYPE_FEDERATED); 553 form, password_manager::CredentialType::CREDENTIAL_TYPE_FEDERATED);
554 EXPECT_CALL(*this, CredentialCallback(credential_info)); 554 EXPECT_CALL(*this, CredentialCallback(credential_info));
555 passwords_data().ChooseCredential( 555 passwords_data().ChooseCredential(
556 form, password_manager::CredentialType::CREDENTIAL_TYPE_PASSWORD); 556 form, password_manager::CredentialType::CREDENTIAL_TYPE_PASSWORD);
557 } 557 }
558 558
559 } // namespace 559 } // namespace
OLDNEW
« no previous file with comments | « chrome/browser/ui/passwords/manage_passwords_state.cc ('k') | chrome/browser/ui/passwords/manage_passwords_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698