Elide text in the new Autofill UI

chrome/browser/ui/autofill/autofill_popup_controller_impl.h

Index: chrome/browser/ui/autofill/autofill_popup_controller_impl.h
diff --git a/chrome/browser/ui/autofill/autofill_popup_controller_impl.h b/chrome/browser/ui/autofill/autofill_popup_controller_impl.h
index 321d56b43c4ce6b2155ee4cbf4363d474a245c24..6e9eebcb138da90ba3b4d0dc3a36b9b508134225 100644
--- a/chrome/browser/ui/autofill/autofill_popup_controller_impl.h
+++ b/chrome/browser/ui/autofill/autofill_popup_controller_impl.h
@@ -128,6 +128,15 @@ class AutofillPopupControllerImpl : public AutofillPopupController,
   virtual void InvalidateRow(size_t row);
 
  private:
+  // Calculate the width of the row, excluding all the text. This provides
+  // the size of the row that won't be reducible (since all the text can be
+  // elided if there isn't enough space).
+  int RowWidthWithoutText(int row);
+
+  // Calculates and sets the width of the popup based on the data to be shown,
+  // the screen size and position of the popup (virtual for testing).
+  virtual int CalculateAndSetPopupWidth();

[Ilya Sherman, 2013/01/11 23:05:35]

IMO the code would be cleaner if this either returned the calculated width or
fully updated the popup_bounds_ directly, rather than partially updating the
popup_bounds_ and also returning the width.  I'd probably recommend the latter
option, and naming this method something like UpdatePopupWidth().

[csharp, 2013/01/14 20:32:33]

Done.

+
   AutofillPopupView* view_;  // Weak reference.
   AutofillPopupDelegate* delegate_;  // Weak reference.
   gfx::NativeView container_view_;  // Weak reference.