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

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

Issue 10987100: Switch AutofillManager to be UserData on WebContents. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix external delegate Created 8 years, 2 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: chrome/browser/autofill/autofill_metrics_unittest.cc
diff --git a/chrome/browser/autofill/autofill_metrics_unittest.cc b/chrome/browser/autofill/autofill_metrics_unittest.cc
index c1d1b00c8841ee4104b0995f7b5446cfdf6f6054..3cadb33c59c637449af8db7430248b1f299b05d6 100644
--- a/chrome/browser/autofill/autofill_metrics_unittest.cc
+++ b/chrome/browser/autofill/autofill_metrics_unittest.cc
@@ -172,10 +172,10 @@ class TestFormStructure : public FormStructure {
class TestAutofillManager : public AutofillManager {
public:
- TestAutofillManager(autofill::AutofillManagerDelegate* manager_delegate,
- TabContents* tab_contents,
+ TestAutofillManager(content::WebContents* web_contents,
+ autofill::AutofillManagerDelegate* manager_delegate,
TestPersonalDataManager* personal_manager)
- : AutofillManager(manager_delegate, tab_contents, personal_manager),
+ : AutofillManager(web_contents, manager_delegate, personal_manager),
autofill_enabled_(true),
did_finish_async_form_submit_(false),
message_loop_is_running_(false) {
@@ -298,8 +298,8 @@ void AutofillMetricsTest::SetUp() {
TabContentsTestHarness::SetUp();
TabAutofillManagerDelegate::CreateForWebContents(web_contents());
autofill_manager_ = new TestAutofillManager(
+ web_contents(),
TabAutofillManagerDelegate::FromWebContents(web_contents()),
- tab_contents(),
&personal_data_);
file_thread_.Start();
« no previous file with comments | « chrome/browser/autofill/autofill_manager_unittest.cc ('k') | chrome/browser/autofill/form_structure_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698