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

Unified Diff: chrome/browser/autofill/autofill_external_delegate_unittest.cc

Issue 10837363: Introduce AutofillClient and use it to get rid of PasswordManager dependency. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix header Created 8 years, 4 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/autofill/api/autofill_manager_delegate.h ('k') | chrome/browser/autofill/autofill_manager.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/autofill/autofill_external_delegate_unittest.cc
diff --git a/chrome/browser/autofill/autofill_external_delegate_unittest.cc b/chrome/browser/autofill/autofill_external_delegate_unittest.cc
index 0e6183d863af1bc94e07ddd45172136bdb50d88f..86ae8eee5c6f1c28a6ef96a9f13f95702c3e8b61 100644
--- a/chrome/browser/autofill/autofill_external_delegate_unittest.cc
+++ b/chrome/browser/autofill/autofill_external_delegate_unittest.cc
@@ -9,6 +9,7 @@
#include "base/string16.h"
#include "chrome/browser/autofill/autofill_manager.h"
#include "chrome/browser/autofill/test_autofill_external_delegate.h"
+#include "chrome/browser/ui/autofill/tab_autofill_manager_delegate.h"
#include "chrome/browser/ui/tab_contents/test_tab_contents.h"
#include "chrome/test/base/testing_profile.h"
#include "content/public/test/test_browser_thread.h"
@@ -65,7 +66,9 @@ class MockAutofillManager : public AutofillManager {
explicit MockAutofillManager(TabContents* tab_contents)
// Force to use the constructor designated for unit test, but we don't
// really need personal_data in this test so we pass a NULL pointer.
- : AutofillManager(tab_contents, NULL) {}
+ : AutofillManager(&delegate_, tab_contents, NULL),
+ delegate_(tab_contents) {
+ }
MOCK_METHOD4(OnFillAutofillFormData,
void(int query_id,
@@ -75,6 +78,9 @@ class MockAutofillManager : public AutofillManager {
protected:
virtual ~MockAutofillManager() {}
+
+ private:
+ TabAutofillManagerDelegate delegate_;
};
} // namespace
« no previous file with comments | « chrome/browser/autofill/api/autofill_manager_delegate.h ('k') | chrome/browser/autofill/autofill_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698