OLD | NEW |
1 // Copyright 2014 The Chromium Authors. All rights reserved. | 1 // Copyright 2014 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 <utility> | 5 #include <utility> |
6 #include <vector> | 6 #include <vector> |
7 | 7 |
8 #include "base/bind.h" | 8 #include "base/bind.h" |
9 #include "base/macros.h" | 9 #include "base/macros.h" |
10 #include "base/memory/scoped_ptr.h" | 10 #include "base/memory/scoped_ptr.h" |
11 #include "base/strings/string_number_conversions.h" | 11 #include "base/strings/string_number_conversions.h" |
12 #include "base/strings/utf_string_conversions.h" | 12 #include "base/strings/utf_string_conversions.h" |
13 #include "build/build_config.h" | 13 #include "build/build_config.h" |
14 #include "chrome/browser/ui/passwords/manage_passwords_bubble_model.h" | 14 #include "chrome/browser/ui/passwords/manage_passwords_bubble_model.h" |
15 #include "chrome/browser/ui/passwords/manage_passwords_icon_view.h" | 15 #include "chrome/browser/ui/passwords/manage_passwords_icon_view.h" |
16 #include "chrome/browser/ui/passwords/manage_passwords_ui_controller_mock.h" | 16 #include "chrome/browser/ui/passwords/manage_passwords_ui_controller_mock.h" |
17 #include "chrome/browser/ui/passwords/password_dialog_controller.h" | 17 #include "chrome/browser/ui/passwords/password_dialog_controller.h" |
18 #include "chrome/browser/ui/passwords/password_dialog_prompts.h" | 18 #include "chrome/browser/ui/passwords/password_dialog_prompts.h" |
19 #include "chrome/test/base/chrome_render_view_host_test_harness.h" | 19 #include "chrome/test/base/chrome_render_view_host_test_harness.h" |
20 #include "chrome/test/base/testing_profile.h" | 20 #include "chrome/test/base/testing_profile.h" |
21 #include "components/autofill/core/common/password_form.h" | 21 #include "components/autofill/core/common/password_form.h" |
22 #include "components/password_manager/core/browser/password_bubble_experiment.h" | 22 #include "components/password_manager/core/browser/password_bubble_experiment.h" |
23 #include "components/password_manager/core/browser/password_form_manager.h" | 23 #include "components/password_manager/core/browser/password_form_manager.h" |
24 #include "components/password_manager/core/browser/password_manager.h" | 24 #include "components/password_manager/core/browser/password_manager.h" |
25 #include "components/password_manager/core/browser/statistics_table.h" | 25 #include "components/password_manager/core/browser/statistics_table.h" |
26 #include "components/password_manager/core/browser/stub_password_manager_client.
h" | 26 #include "components/password_manager/core/browser/stub_password_manager_client.
h" |
27 #include "components/password_manager/core/browser/stub_password_manager_driver.
h" | 27 #include "components/password_manager/core/browser/stub_password_manager_driver.
h" |
28 #include "components/password_manager/core/common/credential_manager_types.h" | |
29 #include "components/password_manager/core/common/password_manager_ui.h" | 28 #include "components/password_manager/core/common/password_manager_ui.h" |
30 #include "components/prefs/pref_service.h" | 29 #include "components/prefs/pref_service.h" |
31 #include "components/variations/variations_associated_data.h" | 30 #include "components/variations/variations_associated_data.h" |
32 #include "content/public/browser/navigation_details.h" | 31 #include "content/public/browser/navigation_details.h" |
33 #include "content/public/test/test_browser_thread_bundle.h" | 32 #include "content/public/test/test_browser_thread_bundle.h" |
34 #include "content/public/test/web_contents_tester.h" | 33 #include "content/public/test/web_contents_tester.h" |
35 #include "testing/gmock/include/gmock/gmock.h" | 34 #include "testing/gmock/include/gmock/gmock.h" |
36 #include "testing/gtest/include/gtest/gtest.h" | 35 #include "testing/gtest/include/gtest/gtest.h" |
37 | 36 |
38 using ::testing::DoAll; | 37 using ::testing::DoAll; |
(...skipping 16 matching lines...) Expand all Loading... |
55 MOCK_METHOD0(ShowAccountChooser, void()); | 54 MOCK_METHOD0(ShowAccountChooser, void()); |
56 MOCK_METHOD0(ShowAutoSigninPrompt, void()); | 55 MOCK_METHOD0(ShowAutoSigninPrompt, void()); |
57 MOCK_METHOD0(ControllerGone, void()); | 56 MOCK_METHOD0(ControllerGone, void()); |
58 | 57 |
59 private: | 58 private: |
60 DISALLOW_COPY_AND_ASSIGN(DialogPromptMock); | 59 DISALLOW_COPY_AND_ASSIGN(DialogPromptMock); |
61 }; | 60 }; |
62 | 61 |
63 class TestManagePasswordsIconView : public ManagePasswordsIconView { | 62 class TestManagePasswordsIconView : public ManagePasswordsIconView { |
64 public: | 63 public: |
65 TestManagePasswordsIconView() {} | 64 TestManagePasswordsIconView() = default; |
66 | 65 |
67 void SetState(password_manager::ui::State state) override { | 66 void SetState(password_manager::ui::State state) override { |
68 state_ = state; | 67 state_ = state; |
69 } | 68 } |
70 password_manager::ui::State state() { return state_; } | 69 password_manager::ui::State state() { return state_; } |
71 | 70 |
72 private: | 71 private: |
73 password_manager::ui::State state_; | 72 password_manager::ui::State state_; |
74 | 73 |
75 DISALLOW_COPY_AND_ASSIGN(TestManagePasswordsIconView); | 74 DISALLOW_COPY_AND_ASSIGN(TestManagePasswordsIconView); |
(...skipping 73 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
149 | 148 |
150 class ManagePasswordsUIControllerTest : public ChromeRenderViewHostTestHarness { | 149 class ManagePasswordsUIControllerTest : public ChromeRenderViewHostTestHarness { |
151 public: | 150 public: |
152 ManagePasswordsUIControllerTest() | 151 ManagePasswordsUIControllerTest() |
153 : password_manager_(&client_), field_trial_list_(nullptr) {} | 152 : password_manager_(&client_), field_trial_list_(nullptr) {} |
154 | 153 |
155 void SetUp() override; | 154 void SetUp() override; |
156 | 155 |
157 autofill::PasswordForm& test_local_form() { return test_local_form_; } | 156 autofill::PasswordForm& test_local_form() { return test_local_form_; } |
158 autofill::PasswordForm& test_federated_form() { return test_federated_form_; } | 157 autofill::PasswordForm& test_federated_form() { return test_federated_form_; } |
159 password_manager::CredentialInfo* credential_info() const { | |
160 return credential_info_.get(); | |
161 } | |
162 DialogPromptMock& dialog_prompt() { return dialog_prompt_; } | 158 DialogPromptMock& dialog_prompt() { return dialog_prompt_; } |
163 | 159 |
164 TestManagePasswordsUIController* controller() { | 160 TestManagePasswordsUIController* controller() { |
165 return static_cast<TestManagePasswordsUIController*>( | 161 return static_cast<TestManagePasswordsUIController*>( |
166 ManagePasswordsUIController::FromWebContents(web_contents())); | 162 ManagePasswordsUIController::FromWebContents(web_contents())); |
167 } | 163 } |
168 | 164 |
169 void CredentialCallback(const password_manager::CredentialInfo& info) { | |
170 credential_info_.reset(new password_manager::CredentialInfo(info)); | |
171 } | |
172 | |
173 void ExpectIconStateIs(password_manager::ui::State state); | 165 void ExpectIconStateIs(password_manager::ui::State state); |
174 void ExpectIconAndControllerStateIs(password_manager::ui::State state); | 166 void ExpectIconAndControllerStateIs(password_manager::ui::State state); |
175 | 167 |
176 scoped_ptr<password_manager::PasswordFormManager> | 168 scoped_ptr<password_manager::PasswordFormManager> |
177 CreateFormManagerWithBestMatches( | 169 CreateFormManagerWithBestMatches( |
178 const autofill::PasswordForm& observed_form, | 170 const autofill::PasswordForm& observed_form, |
179 ScopedVector<autofill::PasswordForm> best_matches); | 171 ScopedVector<autofill::PasswordForm> best_matches); |
180 | 172 |
181 scoped_ptr<password_manager::PasswordFormManager> CreateFormManager(); | 173 scoped_ptr<password_manager::PasswordFormManager> CreateFormManager(); |
182 | 174 |
183 // Tests that the state is not changed when the password is autofilled. | 175 // Tests that the state is not changed when the password is autofilled. |
184 void TestNotChangingStateOnAutofill( | 176 void TestNotChangingStateOnAutofill( |
185 password_manager::ui::State state); | 177 password_manager::ui::State state); |
186 | 178 |
| 179 MOCK_METHOD1(CredentialCallback, void(const autofill::PasswordForm*)); |
| 180 |
187 private: | 181 private: |
188 password_manager::StubPasswordManagerClient client_; | 182 password_manager::StubPasswordManagerClient client_; |
189 password_manager::StubPasswordManagerDriver driver_; | 183 password_manager::StubPasswordManagerDriver driver_; |
190 password_manager::PasswordManager password_manager_; | 184 password_manager::PasswordManager password_manager_; |
191 | 185 |
192 autofill::PasswordForm test_local_form_; | 186 autofill::PasswordForm test_local_form_; |
193 autofill::PasswordForm test_federated_form_; | 187 autofill::PasswordForm test_federated_form_; |
194 scoped_ptr<password_manager::CredentialInfo> credential_info_; | |
195 base::FieldTrialList field_trial_list_; | 188 base::FieldTrialList field_trial_list_; |
196 DialogPromptMock dialog_prompt_; | 189 DialogPromptMock dialog_prompt_; |
197 }; | 190 }; |
198 | 191 |
199 void ManagePasswordsUIControllerTest::SetUp() { | 192 void ManagePasswordsUIControllerTest::SetUp() { |
200 ChromeRenderViewHostTestHarness::SetUp(); | 193 ChromeRenderViewHostTestHarness::SetUp(); |
201 | 194 |
202 // Create the test UIController here so that it's bound to | 195 // Create the test UIController here so that it's bound to |
203 // |test_web_contents_|, and will be retrieved correctly via | 196 // |test_web_contents_|, and will be retrieved correctly via |
204 // ManagePasswordsUIController::FromWebContents in |controller()|. | 197 // ManagePasswordsUIController::FromWebContents in |controller()|. |
(...skipping 314 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
519 controller()->GetState()); | 512 controller()->GetState()); |
520 EXPECT_EQ(origin, controller()->GetOrigin()); | 513 EXPECT_EQ(origin, controller()->GetOrigin()); |
521 EXPECT_THAT(controller()->GetCurrentForms(), | 514 EXPECT_THAT(controller()->GetCurrentForms(), |
522 ElementsAre(Pointee(test_local_form()))); | 515 ElementsAre(Pointee(test_local_form()))); |
523 ASSERT_THAT(dialog_controller->GetLocalForms(), | 516 ASSERT_THAT(dialog_controller->GetLocalForms(), |
524 ElementsAre(Pointee(test_local_form()))); | 517 ElementsAre(Pointee(test_local_form()))); |
525 EXPECT_THAT(dialog_controller->GetFederationsForms(), testing::IsEmpty()); | 518 EXPECT_THAT(dialog_controller->GetFederationsForms(), testing::IsEmpty()); |
526 ExpectIconStateIs(password_manager::ui::INACTIVE_STATE); | 519 ExpectIconStateIs(password_manager::ui::INACTIVE_STATE); |
527 | 520 |
528 EXPECT_CALL(dialog_prompt(), ControllerGone()); | 521 EXPECT_CALL(dialog_prompt(), ControllerGone()); |
| 522 EXPECT_CALL(*this, CredentialCallback(Pointee(test_local_form()))); |
529 dialog_controller->OnChooseCredentials( | 523 dialog_controller->OnChooseCredentials( |
530 *dialog_controller->GetLocalForms()[0], | 524 *dialog_controller->GetLocalForms()[0], |
531 password_manager::CredentialType::CREDENTIAL_TYPE_PASSWORD); | 525 password_manager::CredentialType::CREDENTIAL_TYPE_PASSWORD); |
532 EXPECT_EQ(password_manager::ui::MANAGE_STATE, controller()->GetState()); | 526 EXPECT_EQ(password_manager::ui::MANAGE_STATE, controller()->GetState()); |
533 ASSERT_TRUE(credential_info()); | |
534 EXPECT_EQ(test_local_form().username_value, credential_info()->id); | |
535 EXPECT_EQ(test_local_form().password_value, credential_info()->password); | |
536 EXPECT_TRUE(credential_info()->federation.unique()); | |
537 EXPECT_EQ(password_manager::CredentialType::CREDENTIAL_TYPE_PASSWORD, | |
538 credential_info()->type); | |
539 } | 527 } |
540 | 528 |
541 TEST_F(ManagePasswordsUIControllerTest, ChooseCredentialLocalButFederated) { | 529 TEST_F(ManagePasswordsUIControllerTest, ChooseCredentialLocalButFederated) { |
542 ScopedVector<autofill::PasswordForm> local_credentials; | 530 ScopedVector<autofill::PasswordForm> local_credentials; |
543 local_credentials.push_back( | 531 local_credentials.push_back( |
544 new autofill::PasswordForm(test_federated_form())); | 532 new autofill::PasswordForm(test_federated_form())); |
545 ScopedVector<autofill::PasswordForm> federated_credentials; | 533 ScopedVector<autofill::PasswordForm> federated_credentials; |
546 GURL origin("http://example.com"); | 534 GURL origin("http://example.com"); |
547 PasswordDialogController* dialog_controller = nullptr; | 535 PasswordDialogController* dialog_controller = nullptr; |
548 EXPECT_CALL(*controller(), CreateAccountChooser(_)).WillOnce( | 536 EXPECT_CALL(*controller(), CreateAccountChooser(_)).WillOnce( |
549 DoAll(SaveArg<0>(&dialog_controller), Return(&dialog_prompt()))); | 537 DoAll(SaveArg<0>(&dialog_controller), Return(&dialog_prompt()))); |
550 EXPECT_CALL(dialog_prompt(), ShowAccountChooser()); | 538 EXPECT_CALL(dialog_prompt(), ShowAccountChooser()); |
551 EXPECT_TRUE(controller()->OnChooseCredentials( | 539 EXPECT_TRUE(controller()->OnChooseCredentials( |
552 std::move(local_credentials), std::move(federated_credentials), origin, | 540 std::move(local_credentials), std::move(federated_credentials), origin, |
553 base::Bind(&ManagePasswordsUIControllerTest::CredentialCallback, | 541 base::Bind(&ManagePasswordsUIControllerTest::CredentialCallback, |
554 base::Unretained(this)))); | 542 base::Unretained(this)))); |
555 EXPECT_EQ(password_manager::ui::CREDENTIAL_REQUEST_STATE, | 543 EXPECT_EQ(password_manager::ui::CREDENTIAL_REQUEST_STATE, |
556 controller()->GetState()); | 544 controller()->GetState()); |
557 EXPECT_EQ(origin, controller()->GetOrigin()); | 545 EXPECT_EQ(origin, controller()->GetOrigin()); |
558 EXPECT_THAT(controller()->GetCurrentForms(), | 546 EXPECT_THAT(controller()->GetCurrentForms(), |
559 ElementsAre(Pointee(test_federated_form()))); | 547 ElementsAre(Pointee(test_federated_form()))); |
560 ASSERT_THAT(dialog_controller->GetLocalForms(), | 548 ASSERT_THAT(dialog_controller->GetLocalForms(), |
561 ElementsAre(Pointee(test_federated_form()))); | 549 ElementsAre(Pointee(test_federated_form()))); |
562 EXPECT_THAT(dialog_controller->GetFederationsForms(), testing::IsEmpty()); | 550 EXPECT_THAT(dialog_controller->GetFederationsForms(), testing::IsEmpty()); |
563 ExpectIconStateIs(password_manager::ui::INACTIVE_STATE); | 551 ExpectIconStateIs(password_manager::ui::INACTIVE_STATE); |
564 | 552 |
565 EXPECT_CALL(dialog_prompt(), ControllerGone()); | 553 EXPECT_CALL(dialog_prompt(), ControllerGone()); |
| 554 EXPECT_CALL(*this, CredentialCallback(Pointee(test_federated_form()))); |
566 dialog_controller->OnChooseCredentials( | 555 dialog_controller->OnChooseCredentials( |
567 *dialog_controller->GetLocalForms()[0], | 556 *dialog_controller->GetLocalForms()[0], |
568 password_manager::CredentialType::CREDENTIAL_TYPE_PASSWORD); | 557 password_manager::CredentialType::CREDENTIAL_TYPE_PASSWORD); |
569 EXPECT_EQ(password_manager::ui::MANAGE_STATE, controller()->GetState()); | 558 EXPECT_EQ(password_manager::ui::MANAGE_STATE, controller()->GetState()); |
570 ASSERT_TRUE(credential_info()); | |
571 EXPECT_EQ(test_federated_form().username_value, credential_info()->id); | |
572 EXPECT_EQ(test_federated_form().federation_origin, | |
573 credential_info()->federation); | |
574 EXPECT_TRUE(credential_info()->password.empty()); | |
575 EXPECT_EQ(password_manager::CredentialType::CREDENTIAL_TYPE_FEDERATED, | |
576 credential_info()->type); | |
577 } | |
578 | |
579 TEST_F(ManagePasswordsUIControllerTest, ChooseCredentialFederated) { | |
580 ScopedVector<autofill::PasswordForm> local_credentials; | |
581 ScopedVector<autofill::PasswordForm> federated_credentials; | |
582 federated_credentials.push_back( | |
583 new autofill::PasswordForm(test_local_form())); | |
584 GURL origin("http://example.com"); | |
585 PasswordDialogController* dialog_controller = nullptr; | |
586 EXPECT_CALL(*controller(), CreateAccountChooser(_)).WillOnce( | |
587 DoAll(SaveArg<0>(&dialog_controller), Return(&dialog_prompt()))); | |
588 EXPECT_CALL(dialog_prompt(), ShowAccountChooser()); | |
589 EXPECT_TRUE(controller()->OnChooseCredentials( | |
590 std::move(local_credentials), std::move(federated_credentials), origin, | |
591 base::Bind(&ManagePasswordsUIControllerTest::CredentialCallback, | |
592 base::Unretained(this)))); | |
593 EXPECT_EQ(password_manager::ui::CREDENTIAL_REQUEST_STATE, | |
594 controller()->GetState()); | |
595 EXPECT_EQ(0u, controller()->GetCurrentForms().size()); | |
596 EXPECT_EQ(origin, controller()->GetOrigin()); | |
597 EXPECT_THAT(dialog_controller->GetLocalForms(), testing::IsEmpty()); | |
598 ASSERT_THAT(dialog_controller->GetFederationsForms(), | |
599 ElementsAre(Pointee(test_local_form()))); | |
600 ExpectIconStateIs(password_manager::ui::INACTIVE_STATE); | |
601 | |
602 EXPECT_CALL(dialog_prompt(), ControllerGone()); | |
603 dialog_controller->OnChooseCredentials( | |
604 *dialog_controller->GetFederationsForms()[0], | |
605 password_manager::CredentialType::CREDENTIAL_TYPE_FEDERATED); | |
606 controller()->OnBubbleHidden(); | |
607 EXPECT_EQ(password_manager::ui::MANAGE_STATE, controller()->GetState()); | |
608 ASSERT_TRUE(credential_info()); | |
609 EXPECT_EQ(test_local_form().username_value, credential_info()->id); | |
610 EXPECT_TRUE(credential_info()->password.empty()); | |
611 EXPECT_EQ(password_manager::CredentialType::CREDENTIAL_TYPE_FEDERATED, | |
612 credential_info()->type); | |
613 } | 559 } |
614 | 560 |
615 TEST_F(ManagePasswordsUIControllerTest, ChooseCredentialCancel) { | 561 TEST_F(ManagePasswordsUIControllerTest, ChooseCredentialCancel) { |
616 ScopedVector<autofill::PasswordForm> local_credentials; | 562 ScopedVector<autofill::PasswordForm> local_credentials; |
617 local_credentials.push_back(new autofill::PasswordForm(test_local_form())); | 563 local_credentials.push_back(new autofill::PasswordForm(test_local_form())); |
618 ScopedVector<autofill::PasswordForm> federated_credentials; | 564 ScopedVector<autofill::PasswordForm> federated_credentials; |
619 GURL origin("http://example.com"); | 565 GURL origin("http://example.com"); |
620 PasswordDialogController* dialog_controller = nullptr; | 566 PasswordDialogController* dialog_controller = nullptr; |
621 EXPECT_CALL(*controller(), CreateAccountChooser(_)).WillOnce( | 567 EXPECT_CALL(*controller(), CreateAccountChooser(_)).WillOnce( |
622 DoAll(SaveArg<0>(&dialog_controller), Return(&dialog_prompt()))); | 568 DoAll(SaveArg<0>(&dialog_controller), Return(&dialog_prompt()))); |
623 EXPECT_CALL(dialog_prompt(), ShowAccountChooser()); | 569 EXPECT_CALL(dialog_prompt(), ShowAccountChooser()); |
624 EXPECT_TRUE(controller()->OnChooseCredentials( | 570 EXPECT_TRUE(controller()->OnChooseCredentials( |
625 std::move(local_credentials), std::move(federated_credentials), origin, | 571 std::move(local_credentials), std::move(federated_credentials), origin, |
626 base::Bind(&ManagePasswordsUIControllerTest::CredentialCallback, | 572 base::Bind(&ManagePasswordsUIControllerTest::CredentialCallback, |
627 base::Unretained(this)))); | 573 base::Unretained(this)))); |
628 EXPECT_EQ(password_manager::ui::CREDENTIAL_REQUEST_STATE, | 574 EXPECT_EQ(password_manager::ui::CREDENTIAL_REQUEST_STATE, |
629 controller()->GetState()); | 575 controller()->GetState()); |
630 EXPECT_EQ(origin, controller()->GetOrigin()); | 576 EXPECT_EQ(origin, controller()->GetOrigin()); |
631 | 577 |
632 EXPECT_CALL(dialog_prompt(), ControllerGone()).Times(0); | 578 EXPECT_CALL(dialog_prompt(), ControllerGone()).Times(0); |
| 579 EXPECT_CALL(*this, CredentialCallback(nullptr)); |
633 dialog_controller->OnCloseDialog(); | 580 dialog_controller->OnCloseDialog(); |
634 EXPECT_EQ(password_manager::ui::MANAGE_STATE, controller()->GetState()); | 581 EXPECT_EQ(password_manager::ui::MANAGE_STATE, controller()->GetState()); |
635 ASSERT_TRUE(credential_info()); | |
636 EXPECT_TRUE(credential_info()->federation.unique()); | |
637 EXPECT_TRUE(credential_info()->password.empty()); | |
638 EXPECT_EQ(password_manager::CredentialType::CREDENTIAL_TYPE_EMPTY, | |
639 credential_info()->type); | |
640 } | 582 } |
641 | 583 |
642 TEST_F(ManagePasswordsUIControllerTest, AutoSignin) { | 584 TEST_F(ManagePasswordsUIControllerTest, AutoSignin) { |
643 ScopedVector<autofill::PasswordForm> local_credentials; | 585 ScopedVector<autofill::PasswordForm> local_credentials; |
644 local_credentials.push_back(new autofill::PasswordForm(test_local_form())); | 586 local_credentials.push_back(new autofill::PasswordForm(test_local_form())); |
645 controller()->OnAutoSignin(std::move(local_credentials), | 587 controller()->OnAutoSignin(std::move(local_credentials), |
646 test_local_form().origin); | 588 test_local_form().origin); |
647 EXPECT_EQ(password_manager::ui::AUTO_SIGNIN_STATE, controller()->GetState()); | 589 EXPECT_EQ(password_manager::ui::AUTO_SIGNIN_STATE, controller()->GetState()); |
648 EXPECT_EQ(test_local_form().origin, controller()->GetOrigin()); | 590 EXPECT_EQ(test_local_form().origin, controller()->GetOrigin()); |
649 ASSERT_FALSE(controller()->GetCurrentForms().empty()); | 591 ASSERT_FALSE(controller()->GetCurrentForms().empty()); |
(...skipping 109 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
759 } | 701 } |
760 | 702 |
761 TEST_F(ManagePasswordsUIControllerTest, UpdatePendingStatePasswordAutofilled) { | 703 TEST_F(ManagePasswordsUIControllerTest, UpdatePendingStatePasswordAutofilled) { |
762 TestNotChangingStateOnAutofill( | 704 TestNotChangingStateOnAutofill( |
763 password_manager::ui::PENDING_PASSWORD_UPDATE_STATE); | 705 password_manager::ui::PENDING_PASSWORD_UPDATE_STATE); |
764 } | 706 } |
765 | 707 |
766 TEST_F(ManagePasswordsUIControllerTest, ConfirmationStatePasswordAutofilled) { | 708 TEST_F(ManagePasswordsUIControllerTest, ConfirmationStatePasswordAutofilled) { |
767 TestNotChangingStateOnAutofill(password_manager::ui::CONFIRMATION_STATE); | 709 TestNotChangingStateOnAutofill(password_manager::ui::CONFIRMATION_STATE); |
768 } | 710 } |
OLD | NEW |