Password bubble: ManagePasswordsIconView is now a BubbleIconView.

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

Index: chrome/browser/ui/views/passwords/manage_passwords_bubble_view.h
diff --git a/chrome/browser/ui/views/passwords/manage_passwords_bubble_view.h b/chrome/browser/ui/views/passwords/manage_passwords_bubble_view.h
index 39e1e5057f41ebefb11944967069ac5ad51a7511..07609c24adac1c7601a232a69e2c10629fe9a2eb 100644
--- a/chrome/browser/ui/views/passwords/manage_passwords_bubble_view.h
+++ b/chrome/browser/ui/views/passwords/manage_passwords_bubble_view.h
@@ -7,6 +7,7 @@
 
 #include "base/basictypes.h"
 #include "chrome/browser/ui/passwords/manage_passwords_bubble.h"
+#include "chrome/browser/ui/views/passwords/save_password_refusal_combobox_model.h"
 #include "ui/views/bubble/bubble_delegate.h"
 #include "ui/views/controls/button/button.h"
 #include "ui/views/controls/combobox/combobox.h"
@@ -106,13 +107,18 @@ class ManagePasswordsBubbleView : public ManagePasswordsBubble,
   // shown twice at the same time.
   static ManagePasswordsBubbleView* manage_passwords_bubble_;
 
-  // The buttons that are shown in the bubble.
+  // The views that are shown in the bubble.
   views::BlueButton* save_button_;
-  views::Combobox* refuse_combobox_;
-
   views::Link* manage_link_;
   views::LabelButton* done_button_;
 
+  // The combobox doesn't take ownership of it's model. If we created a combobox

[vabr (Chromium), 2014/04/30 09:01:01]

nit: it's -> its

+  // we need to ensure that we delete the model here, and because the combobox
+  // uses the model in it's destructor, we need to make sure we delete the model
+  // _after_ the combobox itself is deleted.
+  scoped_ptr<SavePasswordRefusalComboboxModel> combobox_model_;
+  scoped_ptr<views::Combobox> refuse_combobox_;
+
   DISALLOW_COPY_AND_ASSIGN(ManagePasswordsBubbleView);
 };