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

Unified Diff: chrome/browser/ui/autofill/autofill_popup_controller_unittest.cc

Issue 306053008: Rename AutofillManagerDelegate to AutofillClient. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 6 years, 6 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/ui/autofill/autofill_popup_controller_unittest.cc
diff --git a/chrome/browser/ui/autofill/autofill_popup_controller_unittest.cc b/chrome/browser/ui/autofill/autofill_popup_controller_unittest.cc
index b320deb399d0aaf814a01e6a6feffc09c3fc549d..e0e4f85138b57726221aa2d6c096bb959b76166f 100644
--- a/chrome/browser/ui/autofill/autofill_popup_controller_unittest.cc
+++ b/chrome/browser/ui/autofill/autofill_popup_controller_unittest.cc
@@ -17,8 +17,8 @@
#include "components/autofill/core/browser/autofill_manager.h"
#include "components/autofill/core/browser/autofill_test_utils.h"
#include "components/autofill/core/browser/popup_item_ids.h"
+#include "components/autofill/core/browser/test_autofill_client.h"
#include "components/autofill/core/browser/test_autofill_external_delegate.h"
-#include "components/autofill/core/browser/test_autofill_manager_delegate.h"
#include "grit/component_scaled_resources.h"
#include "testing/gmock/include/gmock/gmock.h"
#include "testing/gtest/include/gtest/gtest.h"
@@ -53,20 +53,17 @@ class MockAutofillExternalDelegate : public AutofillExternalDelegate {
}
};
-class MockAutofillManagerDelegate
- : public autofill::TestAutofillManagerDelegate {
+class MockAutofillClient : public autofill::TestAutofillClient {
public:
- MockAutofillManagerDelegate()
- : prefs_(autofill::test::PrefServiceForTesting()) {
- }
- virtual ~MockAutofillManagerDelegate() {}
+ MockAutofillClient() : prefs_(autofill::test::PrefServiceForTesting()) {}
+ virtual ~MockAutofillClient() {}
virtual PrefService* GetPrefs() OVERRIDE { return prefs_.get(); }
private:
scoped_ptr<PrefService> prefs_;
- DISALLOW_COPY_AND_ASSIGN(MockAutofillManagerDelegate);
+ DISALLOW_COPY_AND_ASSIGN(MockAutofillClient);
};
class TestAutofillPopupController : public AutofillPopupControllerImpl {
@@ -126,7 +123,7 @@ class TestAutofillPopupController : public AutofillPopupControllerImpl {
class AutofillPopupControllerUnitTest : public ChromeRenderViewHostTestHarness {
public:
AutofillPopupControllerUnitTest()
- : manager_delegate_(new MockAutofillManagerDelegate()),
+ : autofill_client_(new MockAutofillClient()),
autofill_popup_controller_(NULL) {}
virtual ~AutofillPopupControllerUnitTest() {}
@@ -135,7 +132,7 @@ class AutofillPopupControllerUnitTest : public ChromeRenderViewHostTestHarness {
ContentAutofillDriver::CreateForWebContentsAndDelegate(
web_contents(),
- manager_delegate_.get(),
+ autofill_client_.get(),
"en-US",
AutofillManager::ENABLE_AUTOFILL_DOWNLOAD_MANAGER);
ContentAutofillDriver* driver =
@@ -169,7 +166,7 @@ class AutofillPopupControllerUnitTest : public ChromeRenderViewHostTestHarness {
}
protected:
- scoped_ptr<MockAutofillManagerDelegate> manager_delegate_;
+ scoped_ptr<MockAutofillClient> autofill_client_;
scoped_ptr<NiceMock<MockAutofillExternalDelegate> > external_delegate_;
testing::NiceMock<TestAutofillPopupController>* autofill_popup_controller_;
};
« no previous file with comments | « chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc ('k') | chrome/browser/ui/autofill/chrome_autofill_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698