Autofill: Add legal message footer to save credit card bubble.

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/autofill_dialog_types.h"
 #include "chrome/browser/ui/autofill/save_card_bubble_controller.h"
 #include "grit/components_strings.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/views/bubble/bubble_frame_view.h"
 #include "ui/views/controls/button/blue_button.h"
 #include "ui/views/controls/button/label_button.h"
 #include "ui/views/controls/link.h"
+#include "ui/views/controls/styled_label.h"
+#include "ui/views/layout/box_layout.h"
 #include "ui/views/layout/grid_layout.h"
 #include "ui/views/layout/layout_constants.h"
 
 using views::GridLayout;
 
 namespace {
 
 // Fixed width of the column holding the message text.
 const int kWidthOfMessageText = 375;
 
 // TODO(bondd): BubbleManager will eventually move this logic somewhere else,
 // and then kIsOkButtonOnLeftSide can be removed from here and
 // dialog_client_view.cc.
 #if defined(OS_WIN) || defined(OS_CHROMEOS)
 const bool kIsOkButtonOnLeftSide = true;
 #else
 const bool kIsOkButtonOnLeftSide = false;
 #endif
 
+views::StyledLabel* CreateLegalMessageLineLabel(
+    const autofill::SaveCardBubbleController::LegalMessageLine& line,
+    views::StyledLabelListener& listener) {
+  views::StyledLabel* label = new views::StyledLabel(line.text, &listener);
+  const std::vector<autofill::SaveCardBubbleController::LegalMessageLine::Link>&
+      links = line.links;
+  for (size_t i = 0; i < links.size(); ++i) {
+    label->AddStyleRange(links[i].range,
+                         views::StyledLabel::RangeStyleInfo::CreateForLink());
+  }
+  label->SizeToFit(kWidthOfMessageText);
+  return label;
+}
+
 }  // namespace
 
 namespace autofill {
 
 SaveCardBubbleViews::SaveCardBubbleViews(views::View* anchor_view,
                                          content::WebContents* web_contents,
                                          SaveCardBubbleController* controller)
     : LocationBarBubbleDelegateView(anchor_view, web_contents),
       controller_(controller),
       save_button_(nullptr),
@@ skipping 44 matching lines @@
     controller_->OnCancelButton();
   }
   GetWidget()->Close();
 }
 
 void SaveCardBubbleViews::LinkClicked(views::Link* source, int event_flags) {
   DCHECK_EQ(source, learn_more_link_);
   controller_->OnLearnMoreClicked();
 }
 
-void SaveCardBubbleViews::Init() {
+void SaveCardBubbleViews::StyledLabelLinkClicked(views::StyledLabel& label,
+                                                 const gfx::Range& range,
+                                                 int event_flags) {
+  const std::vector<SaveCardBubbleController::LegalMessageLine::Link>& links =
+      legal_message_lines_[&label]->links;
+  for (size_t i = 0; i < links.size(); ++i) {

[Evan Stade, 2015/11/13 01:33:07]

this is why i suggested a map for range->url. But I think you might need to add
operator< for gfx::Range. I guess that's a yak too far for this CL, but it would
be nice to have that.

I do think std::find_if with a lambda would also make this cooler 8)

[bondd, 2015/11/13 22:30:43]

Yep, I tried a map and decided against it when I found out I needed to implement
operator<. Range end can be before start, and there is a "reverse" flag IIRC.
Not crystal clear how operator< would work. Enough people use gfx::Range that it
might drag down into a long discussion (which makes sense, just preferably not
as part of this CL). :)

+    if (links[i].range == range) {
+      controller_->OnLegalMessageLinkClicked(links[i].url);
+      return;
+    }
+  }
+  // |range| was not found.
+  NOTREACHED();
+}
+
+// Create view containing everything except for the footnote.
+// ASCII art diagram of view contents:
+// +---------------------------------------------------------------------------+
+// |                                                                           |
+// |  learn_more_link_                            save_button_ cancel_button_  |
+// |                                                                           |
+// +---------------------------------------------------------------------------+
+views::View* SaveCardBubbleViews::CreateMainContentView() {
   enum {
     COLUMN_SET_ID_MESSAGE,
     COLUMN_SET_ID_BUTTONS,
   };
 
-  GridLayout* layout = new GridLayout(this);
-  SetLayoutManager(layout);
+  View* view = new View();

[Evan Stade, 2015/11/13 01:33:07]

nit: scoped_ptr

[bondd, 2015/11/13 22:30:43]

Sorry, can you explain this out a bit further, or link to an example? I know how
scoped_ptr works, but as I understand it AddChildView() in
SaveCardBubbleViews::Init() will take ownership of this object immediately. I
feel like I'm missing something obvious. :)

[Justin Donnelly, 2015/11/13 22:51:04]

The idea is to be defensive against future changes. We can inspect it now and
say it's correct. But even assuming we're correct, if someone were to come along
later and add an early return to this method then you risk a memory leak.

I think the current school of thought is to avoid handling naked pointers at any
point, no matter how small the scope. (Though this method has plenty of scope
for someone to introduce a leak.)

This requires a few things:

- Declare this as scoped_ptr<View> view(new View()) here.
- Change the return type of this method to scoped_ptr<View>.
- Where you reach the point of the API that insists on a naked pointer
(AddChildView), call release(). In other words:
AddChildView(CreateMainContentView().release());

[bondd, 2015/11/17 00:12:04]

Done. Thanks for the detailed explanation.

+  GridLayout* layout = new GridLayout(view);
+  view->SetLayoutManager(layout);
 
   // Set up ColumnSet that will contain the full-width message text.
   int horizontal_inset = GetBubbleFrameView()->GetTitleInsets().left();
   views::ColumnSet* cs = layout->AddColumnSet(COLUMN_SET_ID_MESSAGE);
   cs->AddPaddingColumn(0, horizontal_inset);
   // TODO(bondd): Current dialog layout has no message text, but future layouts
   // will. This padding column is used until then to set the dialog width.
   cs->AddPaddingColumn(1, kWidthOfMessageText);
   cs->AddPaddingColumn(0, horizontal_inset);
 
   // Set up ColumnSet that will contain the buttons and "learn more" link.
   cs = layout->AddColumnSet(COLUMN_SET_ID_BUTTONS);
   cs->AddPaddingColumn(0, horizontal_inset);
   cs->AddColumn(GridLayout::LEADING, GridLayout::CENTER, 0,
                 GridLayout::USE_PREF, 0, 0);
   cs->AddPaddingColumn(1, 0);
   cs->AddColumn(GridLayout::LEADING, GridLayout::CENTER, 0,
                 GridLayout::USE_PREF, 0, 0);
   cs->AddPaddingColumn(0, views::kRelatedButtonHSpacing);
   cs->AddColumn(GridLayout::LEADING, GridLayout::CENTER, 0,
                 GridLayout::USE_PREF, 0, 0);
   cs->AddPaddingColumn(0, horizontal_inset);
 
   // Create "learn more" link and add it to layout.
   learn_more_link_ = new views::Link(l10n_util::GetStringUTF16(IDS_LEARN_MORE));
+  learn_more_link_->SetUnderline(false);
   learn_more_link_->set_listener(this);
   layout->StartRow(0, COLUMN_SET_ID_BUTTONS);
   layout->AddView(learn_more_link_);
 
   // Create accept button.
   save_button_ = new views::BlueButton(
       this, l10n_util::GetStringUTF16(IDS_AUTOFILL_SAVE_CARD_BUBBLE_ACCEPT));
   save_button_->SetIsDefault(true);
 
   // Create cancel button.
   cancel_button_ = new views::LabelButton(
       this, l10n_util::GetStringUTF16(IDS_AUTOFILL_SAVE_CARD_BUBBLE_DENY));
   cancel_button_->SetStyle(views::Button::STYLE_BUTTON);
 
   if (kIsOkButtonOnLeftSide) {
     layout->AddView(save_button_);
     layout->AddView(cancel_button_);
   } else {
     layout->AddView(cancel_button_);
     layout->AddView(save_button_);
   }
   layout->AddPaddingRow(0, views::kRelatedControlVerticalSpacing);
 
+  return view;
+}
+
+// Create view containing the legal message text.
+views::View* SaveCardBubbleViews::CreateFootnoteView() {
+  // Use BoxLayout to provide insets around the label.
+  View* view = new View();

[Evan Stade, 2015/11/13 01:33:07]

nit: scoped_ptr

+  int horizontal_inset = GetBubbleFrameView()->GetTitleInsets().left();
+  view->SetLayoutManager(
+      new views::BoxLayout(views::BoxLayout::kVertical, horizontal_inset,
+                           views::kRelatedControlVerticalSpacing, 0));
+  view->SetBorder(
+      views::Border::CreateSolidSidedBorder(1, 0, 0, 0, kSubtleBorderColor));
+  view->set_background(
+      views::Background::CreateSolidBackground(kLightShadingColor));
+
+  // Add a StyledLabel for each line of the legal message.
+  for (size_t i = 0; i < controller_->GetLegalMessageLines().size(); ++i) {
+    views::StyledLabel* label = CreateLegalMessageLineLabel(
+        controller_->GetLegalMessageLines()[i], *this);
+    view->AddChildView(label);
+
+    legal_message_lines_[label] = &controller_->GetLegalMessageLines()[i];
+  }
+
+  return view;
+}
+
+void SaveCardBubbleViews::Init() {
+  SetLayoutManager(new views::BoxLayout(views::BoxLayout::kVertical, 0, 0, 0));
+  AddChildView(CreateMainContentView());
+  if (!controller_->GetLegalMessageLines().empty())
+    AddChildView(CreateFootnoteView());
+
   set_margins(gfx::Insets(1, 0, 1, 0));
-  Layout();
 }
 
 }  // namespace autofill