Index: components/autofill/core/browser/autocomplete_history_manager_unittest.cc |
diff --git a/components/autofill/core/browser/autocomplete_history_manager_unittest.cc b/components/autofill/core/browser/autocomplete_history_manager_unittest.cc |
index 6379f072f5ca0f829175f1380ec80e60bf3449d4..3ac46b61bb6e0d7a14ff8809177748e3c0909156 100644 |
--- a/components/autofill/core/browser/autocomplete_history_manager_unittest.cc |
+++ b/components/autofill/core/browser/autocomplete_history_manager_unittest.cc |
@@ -15,8 +15,8 @@ |
#include "components/autofill/core/browser/autofill_external_delegate.h" |
#include "components/autofill/core/browser/autofill_manager.h" |
#include "components/autofill/core/browser/autofill_test_utils.h" |
+#include "components/autofill/core/browser/test_autofill_client.h" |
#include "components/autofill/core/browser/test_autofill_driver.h" |
-#include "components/autofill/core/browser/test_autofill_manager_delegate.h" |
#include "components/autofill/core/browser/webdata/autofill_webdata_service.h" |
#include "components/autofill/core/common/form_data.h" |
#include "components/webdata/common/web_data_service_test_util.h" |
@@ -43,15 +43,12 @@ class MockWebDataService : public AutofillWebDataService { |
virtual ~MockWebDataService() {} |
}; |
-class MockAutofillManagerDelegate |
- : public autofill::TestAutofillManagerDelegate { |
+class MockAutofillClient : public TestAutofillClient { |
public: |
- MockAutofillManagerDelegate( |
- scoped_refptr<MockWebDataService> web_data_service) |
+ MockAutofillClient(scoped_refptr<MockWebDataService> web_data_service) |
: web_data_service_(web_data_service), |
- prefs_(test::PrefServiceForTesting()) { |
- } |
- virtual ~MockAutofillManagerDelegate() {} |
+ prefs_(test::PrefServiceForTesting()) {} |
+ virtual ~MockAutofillClient() {} |
virtual scoped_refptr<AutofillWebDataService> |
GetDatabase() OVERRIDE { return web_data_service_; } |
virtual PrefService* GetPrefs() OVERRIDE { return prefs_.get(); } |
@@ -60,7 +57,7 @@ class MockAutofillManagerDelegate |
scoped_refptr<MockWebDataService> web_data_service_; |
scoped_ptr<PrefService> prefs_; |
- DISALLOW_COPY_AND_ASSIGN(MockAutofillManagerDelegate); |
+ DISALLOW_COPY_AND_ASSIGN(MockAutofillClient); |
}; |
} // namespace |
@@ -71,11 +68,10 @@ class AutocompleteHistoryManagerTest : public testing::Test { |
virtual void SetUp() OVERRIDE { |
web_data_service_ = new MockWebDataService(); |
- manager_delegate_.reset(new MockAutofillManagerDelegate(web_data_service_)); |
+ autofill_client_.reset(new MockAutofillClient(web_data_service_)); |
autofill_driver_.reset(new TestAutofillDriver()); |
- autocomplete_manager_.reset( |
- new AutocompleteHistoryManager(autofill_driver_.get(), |
- manager_delegate_.get())); |
+ autocomplete_manager_.reset(new AutocompleteHistoryManager( |
+ autofill_driver_.get(), autofill_client_.get())); |
} |
virtual void TearDown() OVERRIDE { |
@@ -86,7 +82,7 @@ class AutocompleteHistoryManagerTest : public testing::Test { |
scoped_refptr<MockWebDataService> web_data_service_; |
scoped_ptr<AutocompleteHistoryManager> autocomplete_manager_; |
scoped_ptr<AutofillDriver> autofill_driver_; |
- scoped_ptr<MockAutofillManagerDelegate> manager_delegate_; |
+ scoped_ptr<MockAutofillClient> autofill_client_; |
}; |
// Tests that credit card numbers are not sent to the WebDatabase to be saved. |
@@ -196,9 +192,8 @@ class MockAutofillExternalDelegate : public AutofillExternalDelegate { |
class TestAutocompleteHistoryManager : public AutocompleteHistoryManager { |
public: |
- TestAutocompleteHistoryManager(AutofillDriver* driver, |
- AutofillManagerDelegate* delegate) |
- : AutocompleteHistoryManager(driver, delegate) {} |
+ TestAutocompleteHistoryManager(AutofillDriver* driver, AutofillClient* client) |
+ : AutocompleteHistoryManager(driver, client) {} |
using AutocompleteHistoryManager::SendSuggestions; |
}; |
@@ -208,13 +203,13 @@ class TestAutocompleteHistoryManager : public AutocompleteHistoryManager { |
// Make sure our external delegate is called at the right time. |
TEST_F(AutocompleteHistoryManagerTest, ExternalDelegate) { |
TestAutocompleteHistoryManager autocomplete_history_manager( |
- autofill_driver_.get(), manager_delegate_.get()); |
+ autofill_driver_.get(), autofill_client_.get()); |
- scoped_ptr<AutofillManager> autofill_manager(new AutofillManager( |
- autofill_driver_.get(), |
- manager_delegate_.get(), |
- "en-US", |
- AutofillManager::ENABLE_AUTOFILL_DOWNLOAD_MANAGER)); |
+ scoped_ptr<AutofillManager> autofill_manager( |
+ new AutofillManager(autofill_driver_.get(), |
+ autofill_client_.get(), |
+ "en-US", |
+ AutofillManager::ENABLE_AUTOFILL_DOWNLOAD_MANAGER)); |
MockAutofillExternalDelegate external_delegate(autofill_manager.get(), |
autofill_driver_.get()); |
@@ -228,13 +223,13 @@ TEST_F(AutocompleteHistoryManagerTest, ExternalDelegate) { |
// Verify that no autocomplete suggestion is returned for textarea. |
TEST_F(AutocompleteHistoryManagerTest, NoAutocompleteSuggestionsForTextarea) { |
TestAutocompleteHistoryManager autocomplete_history_manager( |
- autofill_driver_.get(), manager_delegate_.get()); |
+ autofill_driver_.get(), autofill_client_.get()); |
- scoped_ptr<AutofillManager> autofill_manager(new AutofillManager( |
- autofill_driver_.get(), |
- manager_delegate_.get(), |
- "en-US", |
- AutofillManager::ENABLE_AUTOFILL_DOWNLOAD_MANAGER)); |
+ scoped_ptr<AutofillManager> autofill_manager( |
+ new AutofillManager(autofill_driver_.get(), |
+ autofill_client_.get(), |
+ "en-US", |
+ AutofillManager::ENABLE_AUTOFILL_DOWNLOAD_MANAGER)); |
MockAutofillExternalDelegate external_delegate(autofill_manager.get(), |
autofill_driver_.get()); |