Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(8181)

Unified Diff: chrome/browser/ui/views/autofill/generated_credit_card_bubble_views.cc

Issue 21668003: Implement newly saved card bubble for realz and update generated card bubble to (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 7 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/autofill/generated_credit_card_bubble_views.cc
diff --git a/chrome/browser/ui/views/autofill/generated_credit_card_bubble_views.cc b/chrome/browser/ui/views/autofill/generated_credit_card_bubble_views.cc
new file mode 100644
index 0000000000000000000000000000000000000000..53f7f6d6836436c4a565cfa70f1ce4b58c60c583
--- /dev/null
+++ b/chrome/browser/ui/views/autofill/generated_credit_card_bubble_views.cc
@@ -0,0 +1,104 @@
+// Copyright 2013 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/generated_credit_card_bubble_views.h"
+
+#include "chrome/browser/ui/autofill/generated_credit_card_bubble_controller.h"
+#include "chrome/browser/ui/browser_finder.h"
+#include "chrome/browser/ui/views/frame/browser_view.h"
+#include "chrome/browser/ui/views/location_bar/location_bar_view.h"
+#include "ui/gfx/font.h"
+#include "ui/gfx/insets.h"
+#include "ui/gfx/size.h"
+#include "ui/views/bubble/bubble_frame_view.h"
+#include "ui/views/controls/styled_label.h"
+#include "ui/views/layout/box_layout.h"
+#include "ui/views/layout/layout_constants.h"
+#include "ui/views/widget/widget.h"
+
+namespace autofill {
+
+namespace {
+
+// Get the view this bubble will be anchored to via |controller|.
+views::View* GetAnchor(
+ const base::WeakPtr<GeneratedCreditCardBubbleController>& controller) {
Evan Stade 2013/08/08 22:43:20 this doesn't need to be a weak pointer.
Dan Beam 2013/08/09 01:47:58 Done.
+ BrowserView* browser_view = BrowserView::GetBrowserViewForBrowser(
+ chrome::FindBrowserWithWebContents(controller->web_contents()));
+ return browser_view->GetLocationBarView()->generated_credit_card_view();
+}
+
+} // namespace
+
+GeneratedCreditCardBubbleViews::~GeneratedCreditCardBubbleViews() {}
+
+void GeneratedCreditCardBubbleViews::Show() {
+ // TODO(dbeam): investigate why this steals focus from the web contents.
+ views::BubbleDelegateView::CreateBubble(this)->Show();
+
+ // This bubble doesn't render correctly on Windows without calling
+ // |SizeToContents()|. This must be called after showing the widget.
+ SizeToContents();
+}
+
+void GeneratedCreditCardBubbleViews::Hide() {
+ GetWidget()->Close();
+}
+
+bool GeneratedCreditCardBubbleViews::IsHiding() const {
+ return GetWidget() && GetWidget()->IsClosed();
+}
+
+gfx::Size GeneratedCreditCardBubbleViews::GetPreferredSize() {
+ return GetPreferredSizeForTitle();
+}
+
+base::string16 GeneratedCreditCardBubbleViews::GetWindowTitle() const {
+ return controller_->TitleText();
Evan Stade 2013/08/08 22:43:20 check |controller_|
Dan Beam 2013/08/09 01:47:58 Done.
+}
+
+void GeneratedCreditCardBubbleViews::Init() {
+ SetLayoutManager(new views::BoxLayout(views::BoxLayout::kVertical, 0, 0,
+ views::kRelatedControlVerticalSpacing));
+
+ const base::string16& contents_text = controller_->ContentsText();
+ views::StyledLabel* contents = new views::StyledLabel(contents_text, this);
+
+ const std::vector<TextRange>& text_ranges = controller_->ContentsTextRanges();
+ for (size_t i = 0; i < text_ranges.size(); ++i) {
+ views::StyledLabel::RangeStyleInfo style;
+
+ if (text_ranges[i].is_link)
+ style = views::StyledLabel::RangeStyleInfo::CreateForLink();
+ else
+ style.font_style = gfx::Font::BOLD;
+
+ contents->AddStyleRange(text_ranges[i].range, style);
+ }
+
+ AddChildView(contents);
+}
+
+void GeneratedCreditCardBubbleViews::StyledLabelLinkClicked(const ui::Range& r,
+ int event_flags) {
+ if (controller_)
+ controller_->OnLinkClicked();
+}
+
+// static
+base::WeakPtr<GeneratedCreditCardBubble> GeneratedCreditCardBubble::Create(
+ const base::WeakPtr<GeneratedCreditCardBubbleController>& controller) {
+ return (new GeneratedCreditCardBubbleViews(controller))->weak_ptr_factory_.
+ GetWeakPtr();
+}
+
+GeneratedCreditCardBubbleViews::GeneratedCreditCardBubbleViews(
+ const base::WeakPtr<GeneratedCreditCardBubbleController>& controller)
+ : BubbleDelegateView(GetAnchor(controller), views::BubbleBorder::TOP_RIGHT),
+ controller_(controller),
+ weak_ptr_factory_(this) {
+ set_margins(views::BubbleFrameView::GetTitleInsets());
+}
+
+} // namespace autofill

Powered by Google App Engine
This is Rietveld 408576698