Refactoring Manage Passwords Bubble Code to exclude TabSpecificContentSettings

chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc

Index: chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc
diff --git a/chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc b/chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc
index c641a1efa2f8c240ac14319d0d00e524441834d6..d7808848800d632771f4dbef33e8de8dd057fabb 100644
--- a/chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc
+++ b/chrome/browser/ui/views/passwords/manage_passwords_icon_view.cc
@@ -4,6 +4,7 @@
 
 #include "chrome/browser/ui/views/passwords/manage_passwords_icon_view.h"
 
+#include "chrome/browser/ui/passwords/manage_passwords_bubble_ui_controller.h"
 #include "chrome/browser/ui/views/passwords/manage_passwords_bubble_view.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
@@ -22,28 +23,27 @@ ManagePasswordsIconView::ManagePasswordsIconView(
 ManagePasswordsIconView::~ManagePasswordsIconView() {}
 
 void ManagePasswordsIconView::Update(
-    ManagePasswordsIconController* manage_passwords_icon_controller) {
-  SetVisible(
-      manage_passwords_icon_controller &&
-      manage_passwords_icon_controller->manage_passwords_icon_to_be_shown() &&
-      !location_bar_delegate_->GetToolbarModel()->input_in_progress());
+    ManagePasswordsBubbleUIController* manage_passwords_bubble_ui_controller) {
+  SetVisible(manage_passwords_bubble_ui_controller &&
+      manage_passwords_bubble_ui_controller->manage_passwords_icon_to_be_shown()
+      && !location_bar_delegate_->GetToolbarModel()->input_in_progress());

[Garrett Casto, 2013/12/02 22:28:49]

You should be consistent about how you break lines with "&&"
(http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Boolea...).
Personally I would rename "manage_passwords_bubble_ui_controller" to something
shorter ("bubble_ui_controller" maybe?), but if you don't want to do that, I
would think that you would indent like

manage_passwords_bubble_ui_controller->
    manage_passwords_icon_to_be_show() &&
!location_bar_delegate...

[npentrel, 2013/12/03 10:00:25]

Done.

   if (!visible()) {
     ManagePasswordsBubbleView::CloseBubble();
     return;
   }
   SetImage(ui::ResourceBundle::GetSharedInstance().GetImageSkiaNamed(
       IDR_SAVE_PASSWORD));
-  SetTooltip(manage_passwords_icon_controller->password_to_be_saved());
+  SetTooltip(manage_passwords_bubble_ui_controller->password_to_be_saved());
 }
 
 void ManagePasswordsIconView::ShowBubbleIfNeeded(
-    ManagePasswordsIconController* manage_passwords_icon_controller) {
-  if (manage_passwords_icon_controller->
-          manage_passwords_bubble_needs_showing() &&
-      visible() && !ManagePasswordsBubbleView::IsShowing()) {
+    ManagePasswordsBubbleUIController* manage_passwords_bubble_ui_controller) {
+  if (manage_passwords_bubble_ui_controller->
+          manage_passwords_bubble_needs_showing() && visible() &&
+      !ManagePasswordsBubbleView::IsShowing()) {

[Garrett Casto, 2013/12/02 22:28:49]

Break after "&&" here as well. So 

manage_passwords_bubble_ui_controller->
   manage_passwords_bubble_needs_showing() &&
visible() &&
...

[npentrel, 2013/12/03 10:00:25]

Done.

     ManagePasswordsBubbleView::ShowBubble(
         location_bar_delegate_->GetWebContents(), this);
-    manage_passwords_icon_controller->OnBubbleShown();
+    manage_passwords_bubble_ui_controller->OnBubbleShown();
   }
 }