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..217aa75123a0619bf21b5b1ef91cc5bfc60265e7 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
@@ -42,7 +42,11 @@ class ContentSettingImageView : public ui::AnimationDelegate,
   virtual ~ContentSettingImageView();
 
   // Update the decoration from the shown WebContents.

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

Nit: While here: Function comments should be descriptive, not imperative
("Updates", "Performs", not "Update", "Perform") (2 places)

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

Done.

-  void Update(content::WebContents* web_contents);
+  void UpdatePreLayout(content::WebContents* web_contents);
+
+  // Perform any updates which depend on the image having already been laid out
+  // by the owning LocationBarView.
+  void UpdatePostLayout(content::WebContents* web_contents);
 
  private:
   // Number of milliseconds spent animating open; also the time spent animating
@@ -82,6 +86,7 @@ class ContentSettingImageView : public ui::AnimationDelegate,
 
   int GetTotalSpacingWhileAnimating() const;
   void OnClick();
+  void CreateBubble(content::WebContents* web_contents);
 
   LocationBarView* parent_;  // Weak, owns us.
   scoped_ptr<ContentSettingImageModel> content_setting_image_model_;