Autofill: Replace "save credit card" infobar with a bubble (Views only).

chrome/browser/ui/views/location_bar/save_credit_card_icon_view.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/location_bar/save_credit_card_icon_view.h"
+
+#include "chrome/app/chrome_command_ids.h"
+#include "chrome/browser/ui/autofill/save_card_bubble_controller_impl.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "chrome/browser/ui/view_ids.h"
+#include "chrome/browser/ui/views/autofill/save_card_bubble_views.h"
+#include "chrome/grit/generated_resources.h"
+#include "ui/base/l10n/l10n_util.h"
+#include "ui/gfx/vector_icons_public.h"
+
+SaveCreditCardIconView::SaveCreditCardIconView(CommandUpdater* command_updater,
+                                               Browser* browser)
+    : BubbleIconView(command_updater, IDC_SAVE_CREDIT_CARD_FOR_PAGE),
+      browser_(browser) {
+  set_id(VIEW_ID_SAVE_CREDIT_CARD_BUTTON);
+  SetTooltipText(l10n_util::GetStringUTF16(IDS_TOOLTIP_SAVE_CREDIT_CARD));
+  SetToggled(false);
+}
+
+SaveCreditCardIconView::~SaveCreditCardIconView() {}
+
+void SaveCreditCardIconView::SetToggled(bool on) {
+  SetImage(GetVectorIcon(gfx::VectorIconId::AUTOFILL, on));
+}
+
+void SaveCreditCardIconView::OnExecuting(
+    BubbleIconView::ExecuteSource execute_source) {}
+
+views::BubbleDelegateView* SaveCreditCardIconView::GetBubble() const {
+  if (!browser_)
+    return nullptr;
+  content::WebContents* web_contents =
+      browser_->tab_strip_model()->GetActiveWebContents();
+  if (!web_contents)
+    return nullptr;
+  autofill::SaveCardBubbleControllerImpl* controller =
+      autofill::SaveCardBubbleControllerImpl::FromWebContents(web_contents);

[sky, 2015/10/26 20:56:26]

Can this ever be null?

[bondd, 2015/10/27 01:06:39]

controller is created lazily in chrome_autofill_client.cc, so if
ConfirmSaveCreditCard() hasn't been called yet then controller will be null
here. GetBubble() is overriding superclass public member function
BubbleIconView::GetBubble(), so I don't think we have any guarantees that the
bubble will always be shown before this GetBubble() is called.

[sky, 2015/10/27 20:58:25]

Acknowledged.

+  if (!controller)
+    return nullptr;
+
+  return static_cast<autofill::SaveCardBubbleViews*>(
+      controller->save_card_bubble_view());
+}