summaryrefslogtreecommitdiffstats
path: root/chrome/browser/autofill
diff options
context:
space:
mode:
authorcsharp@chromium.org <csharp@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-03-06 19:25:42 +0000
committercsharp@chromium.org <csharp@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-03-06 19:25:42 +0000
commit48c0ffbdd8a7dc47d0c334828e921dddca9bbe8e (patch)
tree0bfd0cde1399cf8a5556c1ff59cd7350ccb96a79 /chrome/browser/autofill
parent908b641b06b63d1f6c9af607ca49fa47b9c0fdb4 (diff)
downloadchromium_src-48c0ffbdd8a7dc47d0c334828e921dddca9bbe8e.zip
chromium_src-48c0ffbdd8a7dc47d0c334828e921dddca9bbe8e.tar.gz
chromium_src-48c0ffbdd8a7dc47d0c334828e921dddca9bbe8e.tar.bz2
Always Close the Autofill UI through the same path
The autofill controller should always handle all hides, to ensure that everything is properly hidden. BUG=178564,178504 Review URL: https://chromiumcodereview.appspot.com/12302034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@186475 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/autofill')
-rw-r--r--chrome/browser/autofill/autofill_external_delegate_browsertest.cc7
1 files changed, 7 insertions, 0 deletions
diff --git a/chrome/browser/autofill/autofill_external_delegate_browsertest.cc b/chrome/browser/autofill/autofill_external_delegate_browsertest.cc
index c7ed801..b04ec0f 100644
--- a/chrome/browser/autofill/autofill_external_delegate_browsertest.cc
+++ b/chrome/browser/autofill/autofill_external_delegate_browsertest.cc
@@ -28,6 +28,9 @@ namespace {
class MockAutofillManagerDelegate
: public autofill::TestAutofillManagerDelegate {
public:
+ MockAutofillManagerDelegate() {}
+ virtual ~MockAutofillManagerDelegate() {}
+
virtual PrefService* GetPrefs() { return &prefs_; }
PrefRegistrySyncable* GetPrefRegistry() {
@@ -46,6 +49,8 @@ class MockAutofillManagerDelegate
private:
TestingPrefServiceSyncable prefs_;
+
+ DISALLOW_COPY_AND_ASSIGN(MockAutofillManagerDelegate);
};
// Subclass AutofillManager so we can create AutofillManager instance.
@@ -60,6 +65,8 @@ class TestAutofillManager : public AutofillManager {
DISALLOW_COPY_AND_ASSIGN(TestAutofillManager);
};
+// Subclass AutofillExternalDelegate so we can create an
+// AutofillExternalDelegate instance.
class TestAutofillExternalDelegate : public AutofillExternalDelegate {
public:
TestAutofillExternalDelegate(content::WebContents* web_contents,