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

components/payments/content/payment_request.cc

Index: components/payments/content/payment_request.cc
diff --git a/components/payments/content/payment_request.cc b/components/payments/content/payment_request.cc
index 15aee2c679179aa5b8350baa43e647edde7dd14a..65d013a88bdc2cef7b8517488aa8be1ef345bb15 100644
--- a/components/payments/content/payment_request.cc
+++ b/components/payments/content/payment_request.cc
@@ -125,11 +125,15 @@ void PaymentRequest::OnPaymentResponseAvailable(
 
 void PaymentRequest::OnShippingOptionIdSelected(
     std::string shipping_option_id) {
+  spec_->StartWaitingForUpdateWith(

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

What do you think about putting this in
PaymentRequestState::SetSelectedShippingOption (before delegate_-> is called)? I
would like to avoid too much UI-type logic in here, and just keep what's
required for the mojo interaction.

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

Yeah makes sense. It was here because of the timer, which doesn't exist anymore.

+      PaymentRequestSpec::UpdateReason::SHIPPING_OPTION);
   client_->OnShippingOptionChange(shipping_option_id);
 }
 
 void PaymentRequest::OnShippingAddressSelected(
     mojom::PaymentAddressPtr address) {
+  spec_->StartWaitingForUpdateWith(

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

Similarly in PaymentRequestState::SetSelectedShippingProfile

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

Done.

+      PaymentRequestSpec::UpdateReason::SHIPPING_ADDRESS);
   client_->OnShippingAddressChange(std::move(address));
 }