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(

[Evan Stade, 2015/11/18 00:21:13]

return scoped_ptr

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

Done.

+    const autofill::SaveCardBubbleController::LegalMessageLine& line,
+    views::StyledLabelListener* listener) {
+  views::StyledLabel* label = new views::StyledLabel(line.text, listener);
+  const std::vector<autofill::SaveCardBubbleController::LegalMessageLine::Link>&

[Evan Stade, 2015/11/18 00:21:13]

this anonymous namespace should be inside the autofill namespace so you don't
need autofill:: qualifiers like this one

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

Done.

+      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 40 matching lines @@
     controller_->OnCancelButton();
   }
   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) {
+  // 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(static_cast<size_t>(label->parent()->child_count()) ==

[Evan Stade, 2015/11/18 00:21:13]

DCHECK_EQ

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

Done.

+         controller_->GetLegalMessageLines().size());
+  int line_index = label->parent()->GetIndexOf(label);

[Evan Stade, 2015/11/18 00:21:13]

don't need this local variable

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

Done. Sometimes I prefer breaking a line up with a local variable like this. IMO
it makes it easier to read, and provides a bit of self-documentation. But you
are the OWNER and that's good enough for me. :)

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

my reasoning is that saving it as a local variable sets up the false expectation
that it's used more than once

+
+  const std::vector<SaveCardBubbleController::LegalMessageLine::Link>& links =
+      controller_->GetLegalMessageLines()[line_index].links;
+  for (size_t i = 0; i < links.size(); ++i) {
+    if (links[i].range == range) {
+      controller_->OnLegalMessageLinkClicked(links[i].url);
+      return;
+    }
+  }

[Evan Stade, 2015/11/18 00:21:13]

nit: \n

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

Done.

+  // |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();
   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.
+scoped_ptr<views::View> SaveCardBubbleViews::CreateFootnoteView() {
+  // Use BoxLayout to provide insets around the label.
+  scoped_ptr<View> view(new View());
+  int horizontal_inset = GetBubbleFrameView()->GetTitleInsets().left();

[Evan Stade, 2015/11/18 00:21:13]

don't need this local var

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

Done.

+  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);

[Evan Stade, 2015/11/18 00:21:13]

no need for a local var for this

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

Done.

+    view->AddChildView(label);
+  }
+
+  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