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

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

Issue 10396003: Add Icon Support for New Autofill Gtk UI (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years, 7 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_popup_view.h
diff --git a/chrome/browser/autofill/autofill_popup_view.h b/chrome/browser/autofill/autofill_popup_view.h
index 7d1771d33c9a29be3ff7f406e8a35ba8700df4bd..367712213f8d479f5ce24ee9172bfe15164c3501 100644
--- a/chrome/browser/autofill/autofill_popup_view.h
+++ b/chrome/browser/autofill/autofill_popup_view.h
@@ -92,6 +92,12 @@ class AutofillPopupView : public content::NotificationObserver {
// have a separator above it.
bool IsSeparatorIndex(int index);
+ // Get the resource value for the given resource, returning -1 if the
+ // resource isn't recognized.
+ int GetIconResourceID(const string16& resource_name);
+
+ bool has_icons() { return has_icons_; }
Ilya Sherman 2012/05/14 20:38:22 nit: No longer needed?
csharp 2012/05/15 13:19:09 Done.
+
private:
// Returns true if the given id refers to an element that can be deleted.
bool CanDelete(int id);
@@ -121,6 +127,10 @@ class AutofillPopupView : public content::NotificationObserver {
// The line that is currently selected by the user.
// |kNoSelection| indicates that no line is currently selected.
int selected_line_;
+
+ // Used to indicate if the current popup has icons for any of
+ // the Autofill data
+ bool has_icons_;
Ilya Sherman 2012/05/14 20:38:22 nit: No longer needed?
csharp 2012/05/15 13:19:09 Done.
};
#endif // CHROME_BROWSER_AUTOFILL_AUTOFILL_POPUP_VIEW_H_
« no previous file with comments | « chrome/browser/autofill/autofill_external_delegate_gtk.cc ('k') | chrome/browser/autofill/autofill_popup_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698