[Web Payments] Add Spinners and timeout while waiting for UpdateWith

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>
@@ skipping 168 matching lines @@
       base::MakeUnique<views::GridLayout>(host);
 
   views::ColumnSet* columns = layout->AddColumnSet(0);
   columns->AddColumn(
       trailing ? views::GridLayout::TRAILING : views::GridLayout::LEADING,
       views::GridLayout::LEADING, 1, views::GridLayout::USE_PREF, 0, 0);
 
   return layout;
 }
 
+std::unique_ptr<views::View> CreateCheckingSpinnerView() {
+  std::unique_ptr<views::View> container = base::MakeUnique<views::View>();
+
+  std::unique_ptr<views::Throbber> throbber =
+      base::MakeUnique<views::Throbber>();
+  throbber->Start();
+  std::unique_ptr<views::GridLayout> layout =
+      base::MakeUnique<views::GridLayout>(container.get());
+  views::ColumnSet* throbber_columns = layout->AddColumnSet(0);
+  throbber_columns->AddPaddingColumn(0.5, 0);
+  throbber_columns->AddColumn(views::GridLayout::Alignment::CENTER,
+                              views::GridLayout::Alignment::TRAILING, 0,
+                              views::GridLayout::SizeType::USE_PREF, 0, 0);
+  throbber_columns->AddPaddingColumn(0.5, 0);
+
+  views::ColumnSet* label_columns = layout->AddColumnSet(1);
+  label_columns->AddPaddingColumn(0.5, 0);
+  label_columns->AddColumn(views::GridLayout::Alignment::CENTER,
+                           views::GridLayout::Alignment::LEADING, 0,
+                           views::GridLayout::SizeType::USE_PREF, 0, 0);
+  label_columns->AddPaddingColumn(0.5, 0);
+
+  layout->StartRow(0.5, 0);
+  layout->AddView(throbber.release());
+
+  layout->StartRow(0.5, 1);
+  layout->AddView(new views::Label(
+      l10n_util::GetStringUTF16(IDS_PAYMENTS_CHECKING_OPTION)));
+
+  container->SetLayoutManager(layout.release());
+
+  return container;
+}
+
 }  // namespace
 
 PaymentSheetViewController::PaymentSheetViewController(
     PaymentRequestSpec* spec,
     PaymentRequestState* state,
     PaymentRequestDialogView* dialog)
     : PaymentRequestSheetController(spec, state, dialog),
       pay_button_(nullptr),
-      widest_name_column_view_width_(ComputeWidestNameColumnViewWidth()) {
+      widest_name_column_view_width_(ComputeWidestNameColumnViewWidth()),
+      current_update_reason_(PaymentRequestSpec::UpdateReason::NONE) {
   spec->AddObserver(this);
   state->AddObserver(this);
 }
 
 PaymentSheetViewController::~PaymentSheetViewController() {
   spec()->RemoveObserver(this);
   state()->RemoveObserver(this);
 }
 
-void PaymentSheetViewController::OnSpecUpdated() {
+void PaymentSheetViewController::OnStartUpdating(
+    PaymentRequestSpec::UpdateReason reason) {
+  current_update_reason_ = reason;
   UpdateContentView();
 }
 
+void PaymentSheetViewController::OnSpecUpdated() {
+  current_update_reason_ = PaymentRequestSpec::UpdateReason::NONE;
+  UpdateContentView();
+}
+
 void PaymentSheetViewController::OnSelectedInformationChanged() {
   UpdatePayButtonState(state()->is_ready_to_pay());
   UpdateContentView();
 }
 
 std::unique_ptr<views::Button>
 PaymentSheetViewController::CreatePrimaryButton() {
   std::unique_ptr<views::Button> button(
       views::MdTextButton::CreateSecondaryUiBlueButton(
           this, l10n_util::GetStringUTF16(IDS_PAYMENTS_PAY_BUTTON)));
@@ skipping 166 matching lines @@
       widest_name_column_view_width_);
   section->set_tag(static_cast<int>(
       PaymentSheetViewControllerTags::SHOW_ORDER_SUMMARY_BUTTON));
   section->set_id(
       static_cast<int>(DialogViewID::PAYMENT_SHEET_SUMMARY_SECTION));
   return section;
 }
 
 std::unique_ptr<views::View>
 PaymentSheetViewController::CreateShippingSectionContent() {
-  auto* profile = state()->selected_shipping_profile();
+  if (current_update_reason_ ==
+      PaymentRequestSpec::UpdateReason::SHIPPING_ADDRESS) {
+    return CreateCheckingSpinnerView();
+  } else {
+    auto* profile = state()->selected_shipping_profile();
 
-  return profile ? GetShippingAddressLabel(AddressStyleType::SUMMARY,
-                                           state()->GetApplicationLocale(),
-                                           *profile)
-                 : base::MakeUnique<views::Label>(base::string16());
+    return profile ? GetShippingAddressLabel(AddressStyleType::SUMMARY,
+                                             state()->GetApplicationLocale(),
+                                             *profile)
+                   : base::MakeUnique<views::Label>(base::string16());
+  }
 }
 
 // Creates the Shipping row, which contains a "Shipping address" label, the
 // user's selected shipping address, and a chevron.
 // +----------------------------------------------+
 // | Shipping Address   Barack Obama              |
 // |                    1600 Pennsylvania Ave.  > |
 // |                    1800MYPOTUS               |
 // +----------------------------------------------+
 std::unique_ptr<views::Button> PaymentSheetViewController::CreateShippingRow() {
@@ skipping 81 matching lines @@
       PaymentSheetViewControllerTags::SHOW_CONTACT_INFO_BUTTON));
   section->set_id(
       static_cast<int>(DialogViewID::PAYMENT_SHEET_CONTACT_INFO_SECTION));
   return section;
 }
 
 std::unique_ptr<views::Button>
 PaymentSheetViewController::CreateShippingOptionRow() {
   mojom::PaymentShippingOption* selected_option =
       spec()->selected_shipping_option();
-  if (!selected_option)
+  if (!selected_option &&
+      current_update_reason_ !=
+          PaymentRequestSpec::UpdateReason::SHIPPING_OPTION) {

[Mathieu, 2017/04/13 15:35:29]

so we'd have to check with UX, but if you select an address
(UpdateReason::SHIPPING_ADDRESS) it would probably make sense to spin this
section too? The updated spec can come back with a different selected option, in
which case it would magically change without a spinner.

[anthonyvd, 2017/04/13 21:05:59]

I'll ask UX and follow up.

     return nullptr;
+  }
 
-  std::unique_ptr<views::View> option_label = CreateShippingOptionLabel(
-      selected_option, selected_option ? spec()->GetFormattedCurrencyAmount(
-                                             selected_option->amount->value)
-                                       : base::ASCIIToUTF16(""));
+  std::unique_ptr<views::View> option_row_content =
+      current_update_reason_ ==
+              PaymentRequestSpec::UpdateReason::SHIPPING_OPTION
+          ? CreateCheckingSpinnerView()
+          : CreateShippingOptionLabel(selected_option,
+                                      selected_option
+                                          ? spec()->GetFormattedCurrencyAmount(
+                                                selected_option->amount->value)
+                                          : base::ASCIIToUTF16(""));
+
   std::unique_ptr<views::Button> section = CreatePaymentSheetRow(
       this, GetShippingOptionSectionString(spec()->shipping_type()),
-      std::move(option_label), std::unique_ptr<views::View>(nullptr),
+      std::move(option_row_content), std::unique_ptr<views::View>(nullptr),
       widest_name_column_view_width_);
   section->set_tag(static_cast<int>(
       PaymentSheetViewControllerTags::SHOW_SHIPPING_OPTION_BUTTON));
   section->set_id(
       static_cast<int>(DialogViewID::PAYMENT_SHEET_SHIPPING_OPTION_SECTION));
   return section;
 }
 
 }  // namespace payments