Upgrade Seperator in New Autofill UI

chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.h

Index: chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.h
diff --git a/chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.h b/chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.h
index 4c13983750fe940ffde438c3765f972232505e25..687609a418ea0e64a3b996155ae9045b0cd4578f 100644
--- a/chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.h
+++ b/chrome/browser/ui/gtk/autofill/autofill_popup_view_gtk.h
@@ -62,15 +62,30 @@ class AutofillPopupViewGtk : public AutofillPopupView,
   // Setup the pango layout to display the autofill results.
   void SetupLayout(const gfx::Rect& window_rect, const GdkColor& text_color);
 
+  // Draw the separator as the given |separator_rect|.
+  void DrawSeparator(cairo_t* cr, const gfx::Rect& separator_rect);

[Ilya Sherman, 2012/05/23 18:58:08]

nit: Is there a more meaningful name we can give to the first parameter?  The
meaning of "cr" is completely unclear to me.

[csharp, 2012/05/24 13:40:10]

Done.

+
+  // Draw the given autofill entry in |entry_rect|.
+  void DrawAutofillEntry(cairo_t* cr,
+                         size_t index,
+                         int actual_content_height,
+                         const gfx::Rect& entry_rect);
+
   // Set the bounds of the popup to show, including the placement of it.
   void SetBounds();
 
   // Get width of popup needed by values.
   int GetPopupRequiredWidth();
 
+  // Get height of popup needed by values.
+  int GetPopupRequiredHeight();
+
   // Convert a y-coordinate to the closest line.
   int LineFromY(int y);
 
+  // Returns the rectangle containing the item at position |index| in the popup.
+  gfx::Rect GetRectForRow(size_t row, int width);
+
   GtkWidget* parent_;  // Weak reference.
   GtkWidget* window_;  // Strong reference.
   PangoLayout* layout_;  // Strong reference