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

Unified Diff: chrome/browser/autofill/api/autofill_manager_delegate.h

Issue 10879037: Rename InfoBarTabService -> InfoBarService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix include guard Created 8 years, 4 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/api/autofill_manager_delegate.h
diff --git a/chrome/browser/autofill/api/autofill_manager_delegate.h b/chrome/browser/autofill/api/autofill_manager_delegate.h
index 3f478a1be042ce1a77bac4a0b5655f45e95bf5d1..47c40dfe08204f7e4dfd6b551d5524c28436f519 100644
--- a/chrome/browser/autofill/api/autofill_manager_delegate.h
+++ b/chrome/browser/autofill/api/autofill_manager_delegate.h
@@ -5,7 +5,7 @@
#ifndef CHROME_BROWSER_AUTOFILL_API_AUTOFILL_MANAGER_DELEGATE_H_
#define CHROME_BROWSER_AUTOFILL_API_AUTOFILL_MANAGER_DELEGATE_H_
-class InfoBarTabService;
+class InfoBarService;
class PrefServiceBase;
namespace autofill {
@@ -23,13 +23,7 @@ class AutofillManagerDelegate {
virtual ~AutofillManagerDelegate() {}
// Gets the infobar service associated with the delegate.
- //
- // TODO(joi): Given the approach (which we will likely use more
- // widely) of a context associated with the instance of the delegate,
- // it seems right to rename InfoBarTabService to just
- // InfoBarService. Naming the getter appropriately, will name the
- // class itself in a follow-up change.
- virtual InfoBarTabService* GetInfoBarService() = 0;
+ virtual InfoBarService* GetInfoBarService() = 0;
// Gets the preferences associated with the delegate.
virtual PrefServiceBase* GetPrefs() = 0;
« no previous file with comments | « chrome/browser/api/infobars/simple_alert_infobar_delegate.cc ('k') | chrome/browser/autofill/autofill_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698