[Payments] Display the shipping option error alongside the address

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

Index: chrome/browser/ui/views/payments/payment_method_view_controller.cc
diff --git a/chrome/browser/ui/views/payments/payment_method_view_controller.cc b/chrome/browser/ui/views/payments/payment_method_view_controller.cc
index 2a2aea501d380dfd1ec735d65dbbbb0c4621a2e3..1062b2da5f219209ea92f43a761dc5841ce26eb8 100644
--- a/chrome/browser/ui/views/payments/payment_method_view_controller.cc
+++ b/chrome/browser/ui/views/payments/payment_method_view_controller.cc
@@ -110,7 +110,12 @@ class PaymentMethodListItem : public payments::PaymentRequestItemList::Item {
     }
   }
 
-  bool CanBeSelected() const override {
+  bool IsDisabled() override {

[anthonyvd, 2017/05/04 13:07:37]

Can this be IsEnabled instead? Affirmative is easier to reason about than
Negative.

[Mathieu, 2017/05/04 13:53:25]

Sure thing!

+    // No items are disabled.
+    return false;
+  }
+
+  bool CanBeSelected() override {
     // If an instrument can't be selected, PerformSelectionFallback is called,
     // where the instrument can be made complete.
     return instrument_->IsCompleteForPayment();