Save password bubble pops up automatically

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

 // Copyright (c) 2012 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_LOCATION_BAR_CONTENT_SETTING_IMAGE_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_CONTENT_SETTING_IMAGE_VIEW_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "chrome/common/content_settings_types.h"
 #include "ui/base/animation/animation_delegate.h"
@@ skipping 23 matching lines @@
                                 public views::WidgetObserver {
  public:
   ContentSettingImageView(ContentSettingsType content_type,
                           LocationBarView* parent,
                           const gfx::Font& font,
                           int font_y_offset,
                           SkColor text_color,
                           SkColor parent_background_color);
   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
   // closed.
   static const int kOpenTimeMS;
 
   // The total animation time, including open and close as well as an
   // intervening "stay open" period.
   static const int kAnimationDurationMS;
 
@@ skipping 19 matching lines @@
 
   // views::WidgetObserver:
   virtual void OnWidgetDestroying(views::Widget* widget) OVERRIDE;
 
   bool background_showing() const {
     return slide_animator_.is_animating() || pause_animation_;
   }
 
   int GetTotalSpacingWhileAnimating() const;
   void OnClick();
+  void CreateBubble(content::WebContents* web_contents);
 
   LocationBarView* parent_;  // Weak, owns us.
   scoped_ptr<ContentSettingImageModel> content_setting_image_model_;
   scoped_ptr<views::Painter> background_painter_;
   views::ImageView* icon_;
   views::Label* text_label_;
   ui::SlideAnimation slide_animator_;
   bool pause_animation_;
   double pause_animation_state_;
   views::Widget* bubble_widget_;
 
   DISALLOW_COPY_AND_ASSIGN(ContentSettingImageView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_CONTENT_SETTING_IMAGE_VIEW_H_