Save password bubble pops up automatically

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

Index: chrome/browser/ui/views/location_bar/content_setting_image_view.h
diff --git a/chrome/browser/ui/views/location_bar/content_setting_image_view.h b/chrome/browser/ui/views/location_bar/content_setting_image_view.h
index 350b3fd760db1df2fb503a94d2f543e31432a9d9..fc1c6dd034c455d66d57e821dfc727bcb6af8e4e 100644
--- a/chrome/browser/ui/views/location_bar/content_setting_image_view.h
+++ b/chrome/browser/ui/views/location_bar/content_setting_image_view.h
@@ -41,8 +41,13 @@ class ContentSettingImageView : public ui::AnimationDelegate,
                           SkColor parent_background_color);
   virtual ~ContentSettingImageView();
 
+  // Updates the decoration from the shown WebContents after layouting has
+  // happened. This is necessary to get the right coordinates for the shown
+  // icons.

[Peter Kasting, 2013/09/03 23:50:43]

This comment seems like it's on the wrong function.

Nit: You might also want something slightly more generic here, and then a more
specific comment in the implementation.  Something like "Perform any updates
which depend on the image having already been laid out by the owning
LocationBarView."

[npentrel, 2013/09/04 09:55:36]

Done.

+  void UpdatePreLayout(content::WebContents* web_contents);
+
   // Update the decoration from the shown WebContents.
-  void Update(content::WebContents* web_contents);
+  void UpdatePostLayout(content::WebContents* web_contents);
 
  private:
   // Number of milliseconds spent animating open; also the time spent animating