Extracting the remaining ManagePasswordsBubbleView::Init code.

chrome/browser/ui/views/passwords/manage_password_item_view.h

Index: chrome/browser/ui/views/passwords/manage_password_item_view.h
diff --git a/chrome/browser/ui/views/passwords/manage_password_item_view.h b/chrome/browser/ui/views/passwords/manage_password_item_view.h
index 5a383e287c40de14c8bf53288b9e0dbfe55b8cd4..a5a0bc86e1084a119d38d5998a3c0dbc73fcde0e 100644
--- a/chrome/browser/ui/views/passwords/manage_password_item_view.h
+++ b/chrome/browser/ui/views/passwords/manage_password_item_view.h
@@ -19,11 +19,14 @@ class ManagePasswordItemView : public views::View,
                                public views::ButtonListener,
                                public views::LinkListener {
  public:
+  enum Position { FIRST_ITEM, MIDDLE_ITEM, LAST_ITEM };

[vabr (Chromium), 2014/04/02 08:38:52]

LAST_ITEM does not seem to be used yet.
Do you have plans to use it soon?
If not, should it be dropped (and MIDDLE_ITEM renamed to denote just the
opposite of FIRST_ITEM)?

[Mike West, 2014/04/02 09:40:06]

Yeah, not sure. I can drop it in this patch, but I think we might need it in a
later patch. *shrug* Easy. :)

+
   ManagePasswordItemView(
       ManagePasswordsBubbleModel* manage_passwords_bubble_model,
       autofill::PasswordForm password_form,
       int field_1_width,
-      int field_2_width);
+      int field_2_width,
+      Position position);
 
   static base::string16 GetPasswordDisplayString(
       const base::string16& password);