Save password functionality added to the save password bubble

chrome/browser/ui/views/content_setting_bubble_contents.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_CONTENT_SETTING_BUBBLE_CONTENTS_H_
 #define CHROME_BROWSER_UI_VIEWS_CONTENT_SETTING_BUBBLE_CONTENTS_H_
 
 #include <map>
 
 #include "base/basictypes.h"
@@ skipping 92 matching lines @@
   content::WebContents* web_contents_;
 
   // A registrar for listening for WEB_CONTENTS_DESTROYED notifications.
   content::NotificationRegistrar registrar_;
 
   // Some of our controls, so we can tell what's been clicked when we get a
   // message.
   PopupLinks popup_links_;
   typedef std::vector<views::RadioButton*> RadioGroup;
   RadioGroup radio_group_;
   views::Link* custom_link_;

[Peter Kasting, 2013/09/09 18:19:02]

Tiny nit: It might be nice to order all these in visual order (L->R, T->B)

[npentrel, 2013/09/09 22:08:34]

They are not always all used. I think it would be more confusing.

[Peter Kasting, 2013/09/10 00:22:11]

OK, let me rephrase:

I would like these to be in some clear order.  Right now they're basically
random.  Most views use visual order, but if there is something that's clearly
better, feel free to use that instead :)

[npentrel, 2013/09/10 16:46:38]

Ok, I changed it to visual order. I thought it was confusing to have label
buttons and then links and then another label button.

   views::Link* manage_link_;
   views::LabelButton* close_button_;
+  views::LabelButton* never_button_;
+  views::LabelButton* save_button_;
   scoped_ptr<views::MenuRunner> menu_runner_;
   MediaMenuPartsMap media_menus_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(ContentSettingBubbleContents);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_CONTENT_SETTING_BUBBLE_CONTENTS_H_