| Index: chrome/browser/autofill/content_autofill_driver_browsertest.cc
|
| diff --git a/chrome/browser/autofill/content_autofill_driver_browsertest.cc b/chrome/browser/autofill/content_autofill_driver_browsertest.cc
|
| index 1df9234914774f652b079779075e9e7763ac7dda..968eb9ecf43df456db2bda631e9accd8cc657451 100644
|
| --- a/chrome/browser/autofill/content_autofill_driver_browsertest.cc
|
| +++ b/chrome/browser/autofill/content_autofill_driver_browsertest.cc
|
| @@ -12,7 +12,7 @@
|
| #include "chrome/test/base/testing_pref_service_syncable.h"
|
| #include "components/autofill/content/browser/content_autofill_driver.h"
|
| #include "components/autofill/core/browser/autofill_manager.h"
|
| -#include "components/autofill/core/browser/test_autofill_manager_delegate.h"
|
| +#include "components/autofill/core/browser/test_autofill_client.h"
|
| #include "content/public/browser/navigation_controller.h"
|
| #include "content/public/browser/page_navigator.h"
|
| #include "content/public/browser/web_contents.h"
|
| @@ -26,11 +26,10 @@
|
| namespace autofill {
|
| namespace {
|
|
|
| -class MockAutofillManagerDelegate
|
| - : public autofill::TestAutofillManagerDelegate {
|
| +class MockAutofillClient : public TestAutofillClient {
|
| public:
|
| - MockAutofillManagerDelegate() {}
|
| - virtual ~MockAutofillManagerDelegate() {}
|
| + MockAutofillClient() {}
|
| + virtual ~MockAutofillClient() {}
|
|
|
| virtual PrefService* GetPrefs() { return &prefs_; }
|
|
|
| @@ -52,7 +51,7 @@ class MockAutofillManagerDelegate
|
| private:
|
| TestingPrefServiceSyncable prefs_;
|
|
|
| - DISALLOW_COPY_AND_ASSIGN(MockAutofillManagerDelegate);
|
| + DISALLOW_COPY_AND_ASSIGN(MockAutofillClient);
|
| };
|
|
|
| // Subclass ContentAutofillDriver so we can create an ContentAutofillDriver
|
| @@ -60,10 +59,10 @@ class MockAutofillManagerDelegate
|
| class TestContentAutofillDriver : public ContentAutofillDriver {
|
| public:
|
| TestContentAutofillDriver(content::WebContents* web_contents,
|
| - AutofillManagerDelegate* delegate)
|
| + AutofillClient* client)
|
| : ContentAutofillDriver(
|
| web_contents,
|
| - delegate,
|
| + client,
|
| g_browser_process->GetApplicationLocale(),
|
| AutofillManager::ENABLE_AUTOFILL_DOWNLOAD_MANAGER) {}
|
| virtual ~TestContentAutofillDriver() {}
|
| @@ -85,10 +84,10 @@ class ContentAutofillDriverBrowserTest : public InProcessBrowserTest,
|
| browser()->tab_strip_model()->GetActiveWebContents();
|
| ASSERT_TRUE(web_contents != NULL);
|
| Observe(web_contents);
|
| - AutofillManager::RegisterProfilePrefs(manager_delegate_.GetPrefRegistry());
|
| + AutofillManager::RegisterProfilePrefs(autofill_client_.GetPrefRegistry());
|
|
|
| autofill_driver_.reset(
|
| - new TestContentAutofillDriver(web_contents, &manager_delegate_));
|
| + new TestContentAutofillDriver(web_contents, &autofill_client_));
|
| }
|
|
|
| // Normally the WebContents will automatically delete the driver, but here
|
| @@ -114,7 +113,7 @@ class ContentAutofillDriverBrowserTest : public InProcessBrowserTest,
|
| base::Closure web_contents_hidden_callback_;
|
| base::Closure nav_entry_committed_callback_;
|
|
|
| - testing::NiceMock<MockAutofillManagerDelegate> manager_delegate_;
|
| + testing::NiceMock<MockAutofillClient> autofill_client_;
|
| scoped_ptr<TestContentAutofillDriver> autofill_driver_;
|
| };
|
|
|
| @@ -122,8 +121,7 @@ IN_PROC_BROWSER_TEST_F(ContentAutofillDriverBrowserTest,
|
| SwitchTabAndHideAutofillPopup) {
|
| // Notification is different on platforms. On linux this will be called twice,
|
| // while on windows only once.
|
| - EXPECT_CALL(manager_delegate_, HideAutofillPopup())
|
| - .Times(testing::AtLeast(1));
|
| + EXPECT_CALL(autofill_client_, HideAutofillPopup()).Times(testing::AtLeast(1));
|
|
|
| scoped_refptr<content::MessageLoopRunner> runner =
|
| new content::MessageLoopRunner;
|
| @@ -139,8 +137,7 @@ IN_PROC_BROWSER_TEST_F(ContentAutofillDriverBrowserTest,
|
| TestPageNavigationHidingAutofillPopup) {
|
| // Notification is different on platforms. On linux this will be called twice,
|
| // while on windows only once.
|
| - EXPECT_CALL(manager_delegate_, HideAutofillPopup())
|
| - .Times(testing::AtLeast(1));
|
| + EXPECT_CALL(autofill_client_, HideAutofillPopup()).Times(testing::AtLeast(1));
|
|
|
| scoped_refptr<content::MessageLoopRunner> runner =
|
| new content::MessageLoopRunner;
|
|
|