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

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

Issue 780183002: Remove ContentPasswordManagerDriver::GetAutofillManager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: vabr review Created 6 years 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/password_form_manager_unittest.cc
diff --git a/components/password_manager/core/browser/password_form_manager_unittest.cc b/components/password_manager/core/browser/password_form_manager_unittest.cc
index 4096d9b2b8a9b9309a287055d2fdc76974a76cde..60d458a73f65706a217062e4ae87ce1911eba0d4 100644
--- a/components/password_manager/core/browser/password_form_manager_unittest.cc
+++ b/components/password_manager/core/browser/password_form_manager_unittest.cc
@@ -74,10 +74,6 @@ class MockPasswordManagerDriver : public StubPasswordManagerDriver {
MOCK_METHOD1(AllowPasswordGenerationForForm,
void(const autofill::PasswordForm&));
- virtual autofill::AutofillManager* GetAutofillManager() override {
- return &mock_autofill_manager_;
- }
-
MockAutofillManager* mock_autofill_manager() {
return &mock_autofill_manager_;
}
@@ -114,6 +110,10 @@ class TestPasswordManagerClient : public StubPasswordManagerClient {
MockPasswordManagerDriver* mock_driver() { return &driver_; }
+ virtual autofill::AutofillManager* GetAutofillManagerForMainFrame() override {
+ return mock_driver()->mock_autofill_manager();
+ }
+
private:
autofill::PasswordForm form_to_filter_;

Powered by Google App Engine
This is Rietveld 408576698