[Web Payments] Add data attribution string to dialog

chrome/browser/ui/views/payments/payment_sheet_view_controller.cc

 // Copyright 2016 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/payments/payment_sheet_view_controller.h"
 
 #include <algorithm>
 #include <memory>
 #include <string>
 #include <utility>
 #include <vector>
 
 #include "base/i18n/message_formatter.h"
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
+#include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/chrome_pages.h"
 #include "chrome/browser/ui/views/payments/payment_request_dialog_view.h"
 #include "chrome/browser/ui/views/payments/payment_request_dialog_view_ids.h"
 #include "chrome/browser/ui/views/payments/payment_request_row_view.h"
 #include "chrome/browser/ui/views/payments/payment_request_views_util.h"
 #include "chrome/grit/generated_resources.h"
 #include "components/autofill/core/browser/field_types.h"
 #include "components/autofill/core/browser/personal_data_manager.h"
+#include "components/payments/content/payment_prefs.h"
 #include "components/payments/content/payment_request_spec.h"
 #include "components/payments/content/payment_request_state.h"
 #include "components/payments/core/currency_formatter.h"
 #include "components/payments/core/payment_instrument.h"
 #include "components/payments/core/strings_util.h"
+#include "components/prefs/pref_service.h"
 #include "components/strings/grit/components_strings.h"
 #include "content/public/browser/web_contents.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/color_utils.h"
 #include "ui/gfx/font.h"
 #include "ui/gfx/paint_vector_icon.h"
 #include "ui/gfx/range/range.h"
 #include "ui/gfx/text_elider.h"
 #include "ui/gfx/text_utils.h"
+#include "ui/views/border.h"
 #include "ui/views/controls/button/md_text_button.h"
 #include "ui/views/controls/image_view.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/controls/styled_label.h"
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/layout/fill_layout.h"
 #include "ui/views/layout/grid_layout.h"
 #include "ui/views/vector_icons.h"
 #include "ui/views/view.h"
 
@@ skipping 401 matching lines @@
       layout->AddView(shipping_row.release());
     }
   }
   layout->StartRow(0, 0);
   layout->AddView(CreatePaymentMethodRow().release());
   if (spec()->request_payer_name() || spec()->request_payer_email() ||
       spec()->request_payer_phone()) {
     layout->StartRow(0, 0);
     layout->AddView(CreateContactInfoRow().release());
   }
+  layout->StartRow(0, 0);
+  layout->AddView(CreateDataSourceRow().release());
 }
 
 // Adds the product logo to the footer.
 // +---------------------------------------------------------+
 // | (•) chrome                               | PAY | CANCEL |
 // +---------------------------------------------------------+
 std::unique_ptr<views::View>
 PaymentSheetViewController::CreateExtraFooterView() {
   return CreateProductLogoFooterView();
 }
@@ skipping 52 matching lines @@
         PaymentSheetViewControllerTags::SHOW_SHIPPING_OPTION_BUTTON):
       dialog()->ShowShippingOptionSheet();
       break;
 
     default:
       PaymentRequestSheetController::ButtonPressed(sender, event);
       break;
   }
 }
 
+void PaymentSheetViewController::StyledLabelLinkClicked(
+    views::StyledLabel* label,
+    const gfx::Range& range,
+    int event_flags) {
+  // The only thing that can trigger this is the user clicking on the "settings"
+  // link in the data attribution text.
+  chrome::ShowSettingsSubPageForProfile(
+      Profile::FromBrowserContext(state()->GetBrowserContext()), "");

[Mathieu, 2017/05/11 00:28:04]

could we expose BrowserContext from the PaymentRequestDialogView perhaps? 

request()->web_contents()->GetBrowserContext()

Would avoid going to the state, which has become a grab bag (most of it my
fault!)

[anthonyvd, 2017/05/11 00:56:46]

Good point, somehow I missed the fact that the dialog could go through |request|
to get the web contents. This also makes it possible to have
PaymentRequestDialogView::GetProfile instead of getting the profile from the
BrowserContext everywhere since PaymentRequestDialogView is in chrome/browser.

+}
+
 void PaymentSheetViewController::UpdatePayButtonState(bool enabled) {
   pay_button_->SetEnabled(enabled);
 }
 
 // Creates the Order Summary row, which contains an "Order Summary" label,
 // an inline list of display items, a Total Amount label, and a Chevron.
 // +----------------------------------------------+
 // | Order Summary   Item 1            $ 1.34     |
 // |                 Item 2            $ 2.00   > |
 // |                 2 more items...              |
@@ skipping 345 matching lines @@
     }
   } else {
     return builder
         .Id(DialogViewID::PAYMENT_SHEET_SHIPPING_OPTION_SECTION_BUTTON)
         .CreateWithButton(
             l10n_util::GetStringUTF16(IDS_PAYMENT_REQUEST_CHOOSE_AN_ADDRESS),
             l10n_util::GetStringUTF16(IDS_CHOOSE), /*button_enabled=*/false);
   }
 }
 
+std::unique_ptr<views::View> PaymentSheetViewController::CreateDataSourceRow() {
+  std::unique_ptr<views::View> content_view = base::MakeUnique<views::View>();
+  views::BoxLayout* layout = new views::BoxLayout(
+      views::BoxLayout::kVertical, kPaymentRequestRowHorizontalInsets, 0, 0);
+  layout->set_main_axis_alignment(views::BoxLayout::MAIN_AXIS_ALIGNMENT_START);
+  layout->set_cross_axis_alignment(
+      views::BoxLayout::CROSS_AXIS_ALIGNMENT_START);
+  content_view->SetLayoutManager(layout);
+
+  base::string16 data_source;
+  bool first_transaction_completed =
+      Profile::FromBrowserContext(state()->GetBrowserContext())
+          ->GetPrefs()
+          ->GetBoolean(payments::kPaymentsFirstTransactionCompleted);
+  if (first_transaction_completed) {

[Mathieu, 2017/05/11 00:28:04]

would benefit from a top-level comment to explain this if-else if-else business

[anthonyvd, 2017/05/11 00:56:46]

Done.

+    data_source =
+        l10n_util::GetStringUTF16(IDS_PAYMENTS_CARD_AND_ADDRESS_SETTINGS);
+  } else {
+    std::string user_email = state()->GetAuthenticatedEmail();
+    if (!user_email.empty()) {
+      // Insert the user's email into the format string.
+      data_source = base::UTF8ToUTF16(base::StringPrintf(
+          l10n_util::GetStringUTF8(
+              IDS_PAYMENTS_CARD_AND_ADDRESS_SETTINGS_SIGNED_IN)
+              .c_str(),
+          user_email.c_str()));
+      // Settings link.

[Mathieu, 2017/05/11 00:28:04]

remove

[anthonyvd, 2017/05/11 00:56:46]

Done.

+    } else {
+      data_source = l10n_util::GetStringUTF16(
+          IDS_PAYMENTS_CARD_AND_ADDRESS_SETTINGS_SIGNED_OUT);
+    }
+  }
+
+  base::string16 begin_tag = base::UTF8ToUTF16("BEGIN_LINK");

[Mathieu, 2017/05/11 00:28:04]

// Activate the "Settings" link in the data source string. 

[anthonyvd, 2017/05/11 00:56:46]

Done.

+  base::string16 end_tag = base::UTF8ToUTF16("END_LINK");
+  size_t link_begin = data_source.find(begin_tag);
+  DCHECK(link_begin != base::string16::npos);
+
+  size_t link_end = data_source.find(end_tag);
+  DCHECK(link_end != base::string16::npos);
+
+  size_t link_length = link_end - link_begin - begin_tag.size();
+  data_source.erase(link_end, end_tag.size());
+  data_source.erase(link_begin, begin_tag.size());
+
+  std::unique_ptr<views::StyledLabel> data_source_label =
+      base::MakeUnique<views::StyledLabel>(data_source, this);
+  data_source_label->SetBorder(views::CreateEmptyBorder(22, 0, 0, 0));
+
+  views::StyledLabel::RangeStyleInfo default_style;
+  default_style.color = data_source_label->GetNativeTheme()->GetSystemColor(
+      ui::NativeTheme::kColorId_LabelDisabledColor);
+  data_source_label->SetDefaultStyle(default_style);
+  data_source_label->AddStyleRange(
+      gfx::Range(link_begin, link_begin + link_length),
+      views::StyledLabel::RangeStyleInfo::CreateForLink());
+  data_source_label->SizeToFit(0);
+  content_view->AddChildView(data_source_label.release());
+  return content_view;
+}
+
 }  // namespace payments