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

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: Created 8 years, 3 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 8e31f3316e4ca05f1d66aea0c86d677af77c9f0a..12b661dadeff7b93294269bc6a09cb8ad206923f 100644
--- a/chrome/browser/autofill/autofill_metrics_unittest.cc
+++ b/chrome/browser/autofill/autofill_metrics_unittest.cc
@@ -174,10 +174,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) {
@@ -300,8 +300,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();

Powered by Google App Engine
This is Rietveld 408576698