Save password bubble pops up automatically

chrome/browser/ui/views/location_bar/location_bar_view.h

Index: chrome/browser/ui/views/location_bar/location_bar_view.h
diff --git a/chrome/browser/ui/views/location_bar/location_bar_view.h b/chrome/browser/ui/views/location_bar/location_bar_view.h
index f846263ee5ca6f42175fc61baafb1a53c878f243..7c1d2e2f831a57d67e9fbc2bd8924e0bb1ec2eae 100644
--- a/chrome/browser/ui/views/location_bar/location_bar_view.h
+++ b/chrome/browser/ui/views/location_bar/location_bar_view.h
@@ -385,7 +385,10 @@ class LocationBarView : public LocationBar,
 
   // Update the visibility state of the Content Blocked icons to reflect what is

[Peter Kasting, 2013/09/05 06:55:19]

Nit: Update -> Updates

[npentrel, 2013/09/05 08:28:09]

Done.

   // actually blocked on the current page.

[Peter Kasting, 2013/09/05 06:55:19]

Nit: Add to end:

Calling this function should always eventually be followed by calling Layout()
and then UpdateContentSettingViewsPostLayout(), to ensure the icons can
completely update their state.

[npentrel, 2013/09/05 08:28:09]

Done.

-  void RefreshContentSettingViews();
+  void UpdateContentSettingViewsPreLayout();
+
+  // Updates after the correct screen coordinates have been set for icons.

[Peter Kasting, 2013/09/05 06:55:19]

Nit: How about:

Allows content setting icons to perform any updating which can't complete until
after the icons have been correctly laid out.  This should be called after
UpdateContentSettingViewsPreLayout() and a subsequent Layout().

[npentrel, 2013/09/05 08:28:09]

Done.

+  void UpdateContentSettingViewsPostLayout();
 
   // Delete all page action views that we have created.
   void DeletePageActionViews();