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

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

Issue 11185011: Remove TabContents from autofill unittests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: remove some Created 8 years, 2 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/autofill_manager.h
diff --git a/chrome/browser/autofill/autofill_manager.h b/chrome/browser/autofill/autofill_manager.h
index e4d2dd135c3829930afdc779bbf1cdd9fd2f94cc..750c7ca637c8d024993fadee542a606d9a937428 100644
--- a/chrome/browser/autofill/autofill_manager.h
+++ b/chrome/browser/autofill/autofill_manager.h
@@ -77,13 +77,8 @@ class AutofillManager : public content::NotificationObserver,
// Registers our Enable/Disable Autofill pref.
static void RegisterUserPrefs(PrefServiceBase* prefs);
- // Set our external delegate.
- // TODO(jrg): consider passing delegate into the ctor. That won't
- // work if the delegate has a pointer to the AutofillManager, but
- // future directions may not need such a pointer.
- void SetExternalDelegate(AutofillExternalDelegate* delegate) {
- external_delegate_ = delegate;
- }
+ // Sets an external delegate.
+ void SetExternalDelegate(AutofillExternalDelegate* delegate);
// Used to say if this class has an external delegate that it is using.
bool HasExternalDelegate();

Powered by Google App Engine
This is Rietveld 408576698