Update location bar bubble/icon behavior.

chrome/browser/ui/views/autofill/save_card_bubble_views.cc

 // Copyright 2015 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.
 
 #include "chrome/browser/ui/views/autofill/save_card_bubble_views.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/ui/autofill/save_card_bubble_controller.h"
 #include "grit/components_strings.h"
 #include "ui/base/l10n/l10n_util.h"
@@ skipping 31 matching lines @@
       controller_(controller),
       save_button_(nullptr),
       cancel_button_(nullptr),
       learn_more_link_(nullptr) {
   DCHECK(controller);
   views::BubbleDelegateView::CreateBubble(this);
 }
 
 SaveCardBubbleViews::~SaveCardBubbleViews() {}
 
-void SaveCardBubbleViews::Show() {
-  GetWidget()->Show();
+void SaveCardBubbleViews::Show(bool user_action) {
+  ShowForReason(user_action ? USER_ACTION : AUTOMATIC);
 }
 
-void SaveCardBubbleViews::Close() {
-  GetWidget()->Close();
-}
-
-void SaveCardBubbleViews::ControllerGone() {
+void SaveCardBubbleViews::Hide() {
   controller_ = nullptr;
-  GetWidget()->Close();
+  Close();
 }
 
 views::View* SaveCardBubbleViews::GetInitiallyFocusedView() {
   return save_button_;
 }
 
 base::string16 SaveCardBubbleViews::GetWindowTitle() const {
   return l10n_util::GetStringUTF16(IDS_AUTOFILL_SAVE_CARD_BUBBLE_TITLE_LOCAL);
 }
 
 bool SaveCardBubbleViews::ShouldShowWindowTitle() const {
   return true;
 }
 
 void SaveCardBubbleViews::WindowClosing() {
   if (controller_)
     controller_->OnBubbleClosed();
 }
 
 void SaveCardBubbleViews::ButtonPressed(views::Button* sender,
                                         const ui::Event& event) {
   if (sender == save_button_) {
     controller_->OnSaveButton();
   } else {
     DCHECK_EQ(sender, cancel_button_);
     controller_->OnCancelButton();
   }
   GetWidget()->Close();

[Peter Kasting, 2015/11/10 00:24:17]

Seems like this should be calling Close() instead.

[Evan Stade, 2015/11/10 19:18:26]

Done.

 }
 
 void SaveCardBubbleViews::LinkClicked(views::Link* source, int event_flags) {
   DCHECK_EQ(source, learn_more_link_);
   controller_->OnLearnMoreClicked();
 }
 
 void SaveCardBubbleViews::Init() {
   enum {
     COLUMN_SET_ID_MESSAGE,
@@ skipping 48 matching lines @@
     layout->AddView(cancel_button_);
     layout->AddView(save_button_);
   }
   layout->AddPaddingRow(0, views::kRelatedControlVerticalSpacing);
 
   set_margins(gfx::Insets(1, 0, 1, 0));
   Layout();
 }
 
 }  // namespace autofill