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

Unified Diff: chrome/browser/autofill/test_autofill_external_delegate.h

Issue 12340065: Move the UI related code from AutofillExternalDelegate to AutofillManagerDelegate (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Try to fix AutofillTest.DisableAutocompleteWhileFilling Created 7 years, 10 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/test_autofill_external_delegate.h
diff --git a/chrome/browser/autofill/test_autofill_external_delegate.h b/chrome/browser/autofill/test_autofill_external_delegate.h
index 10a845f7c7fd7503373974caea37ea11440b7297..19e147345ffecda0f9c176cfface51357f25de3e 100644
--- a/chrome/browser/autofill/test_autofill_external_delegate.h
+++ b/chrome/browser/autofill/test_autofill_external_delegate.h
@@ -16,29 +16,6 @@ namespace autofill {
void GenerateTestAutofillPopup(
AutofillExternalDelegate* autofill_external_delegate);
-// This test class is meant to give tests a base AutofillExternalDelegate
-// class that requires no additional work to compile with (i.e. all the
-// pure virtual functions have been giving empty methods).
-class TestAutofillExternalDelegate : public AutofillExternalDelegate {
- public:
- TestAutofillExternalDelegate(content::WebContents* web_contents,
- AutofillManager* autofill_manager);
- virtual ~TestAutofillExternalDelegate();
-
- virtual void ApplyAutofillSuggestions(
- const std::vector<string16>& autofill_values,
- const std::vector<string16>& autofill_labels,
- const std::vector<string16>& autofill_icons,
- const std::vector<int>& autofill_unique_ids) OVERRIDE;
-
- virtual void HideAutofillPopup() OVERRIDE;
-
- virtual void EnsurePopupForElement(const gfx::RectF& element_bounds) OVERRIDE;
-
- private:
- DISALLOW_COPY_AND_ASSIGN(TestAutofillExternalDelegate);
-};
-
} // namespace autofill
#endif // CHROME_BROWSER_AUTOFILL_TEST_AUTOFILL_EXTERNAL_DELEGATE_H_
« no previous file with comments | « chrome/browser/autofill/autofill_manager_unittest.cc ('k') | chrome/browser/autofill/test_autofill_external_delegate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698