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

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

Index: chrome/browser/ui/views/autofill/save_card_bubble_views.cc
diff --git a/chrome/browser/ui/views/autofill/save_card_bubble_views.cc b/chrome/browser/ui/views/autofill/save_card_bubble_views.cc
index 64d7ae5915fc910753422c294294510b1c10c552..a6d2b384050361f9db43116fed414a36d3b6777e 100644
--- a/chrome/browser/ui/views/autofill/save_card_bubble_views.cc
+++ b/chrome/browser/ui/views/autofill/save_card_bubble_views.cc
@@ -5,6 +5,7 @@
 #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"
@@ -12,11 +13,15 @@
 #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 autofill {
+

[bondd, 2015/11/18 01:59:40]

Other autofill files seem evenly split on whether there should be a blank line
between these two:
namespace autofill {
namespace {

 namespace {
 
 // Fixed width of the column holding the message text.
@@ -31,9 +36,22 @@ const bool kIsOkButtonOnLeftSide = true;
 const bool kIsOkButtonOnLeftSide = false;
 #endif
 
-}  // namespace
+scoped_ptr<views::StyledLabel> CreateLegalMessageLineLabel(
+    const SaveCardBubbleController::LegalMessageLine& line,
+    views::StyledLabelListener* listener) {
+  scoped_ptr<views::StyledLabel> label(
+      new views::StyledLabel(line.text, listener));
+  const std::vector<SaveCardBubbleController::LegalMessageLine::Link>& links =
+      line.links;
+  for (size_t i = 0; i < links.size(); ++i) {

[Evan Stade, 2015/11/18 22:01:19]

for (const SaveCardBubbleController::LegalMessageLine::Link& link : line.links)

[bondd, 2015/11/18 23:41:42]

Done.

+    label->AddStyleRange(links[i].range,
+                         views::StyledLabel::RangeStyleInfo::CreateForLink());
+  }
+  label->SizeToFit(kWidthOfMessageText);
+  return label;
+}
 
-namespace autofill {
+}  // namespace
 
 SaveCardBubbleViews::SaveCardBubbleViews(views::View* anchor_view,
                                          content::WebContents* web_contents,
@@ -91,14 +109,39 @@ void SaveCardBubbleViews::LinkClicked(views::Link* source, int event_flags) {
   controller_->OnLearnMoreClicked();
 }
 
-void SaveCardBubbleViews::Init() {
+void SaveCardBubbleViews::StyledLabelLinkClicked(views::StyledLabel* label,
+                                                 const gfx::Range& range,
+                                                 int event_flags) {
+  // Index of |label| within its parent's view hierarchy is the same as the
+  // legal message line index. DCHECK this assumption to guard against future
+  // layout changes.
+  DCHECK_EQ(static_cast<size_t>(label->parent()->child_count()),
+            controller_->GetLegalMessageLines().size());
+
+  const std::vector<SaveCardBubbleController::LegalMessageLine::Link>& links =

[Evan Stade, 2015/11/18 22:01:19]

I'd say this is a good candidate for auto

const auto& links = foo;
for (const SaveCardBubbleController::LegalMessageLine::Link& link : links) {

}

[bondd, 2015/11/18 23:41:42]

Done.

+      controller_->GetLegalMessageLines()[label->parent()->GetIndexOf(label)]
+          .links;
+  for (size_t i = 0; i < links.size(); ++i) {
+    if (links[i].range == range) {
+      controller_->OnLegalMessageLinkClicked(links[i].url);
+      return;
+    }
+  }
+
+  // |range| was not found.
+  NOTREACHED();
+}
+
+// Create view containing everything except for the footnote.
+scoped_ptr<views::View> SaveCardBubbleViews::CreateMainContentView() {
   enum {
     COLUMN_SET_ID_MESSAGE,
     COLUMN_SET_ID_BUTTONS,
   };
 
-  GridLayout* layout = new GridLayout(this);
-  SetLayoutManager(layout);
+  scoped_ptr<View> view(new View());
+  GridLayout* layout = new GridLayout(view.get());
+  view->SetLayoutManager(layout);
 
   // Set up ColumnSet that will contain the full-width message text.
   int horizontal_inset = GetBubbleFrameView()->GetTitleInsets().left();
@@ -124,6 +167,7 @@ void SaveCardBubbleViews::Init() {
 
   // 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_);
@@ -147,8 +191,39 @@ void SaveCardBubbleViews::Init() {
   }
   layout->AddPaddingRow(0, views::kRelatedControlVerticalSpacing);
 
+  return view;
+}
+
+// Create view containing the legal message text.
+scoped_ptr<views::View> SaveCardBubbleViews::CreateFootnoteView() {
+  // Use BoxLayout to provide insets around the label.
+  scoped_ptr<View> view(new View());
+  view->SetLayoutManager(
+      new views::BoxLayout(views::BoxLayout::kVertical,
+                           GetBubbleFrameView()->GetTitleInsets().left(),
+                           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) {
+    view->AddChildView(CreateLegalMessageLineLabel(
+                           controller_->GetLegalMessageLines()[i], this)
+                           .release());
+  }
+
+  return view;
+}
+
+void SaveCardBubbleViews::Init() {
+  SetLayoutManager(new views::BoxLayout(views::BoxLayout::kVertical, 0, 0, 0));
+  AddChildView(CreateMainContentView().release());
+  if (!controller_->GetLegalMessageLines().empty())
+    AddChildView(CreateFootnoteView().release());
+
   set_margins(gfx::Insets(1, 0, 1, 0));
-  Layout();
 }
 
 }  // namespace autofill