[Payments] Display the shipping option error alongside the address

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

 // Copyright 2017 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_method_view_controller.h"
 
 #include <memory>
 #include <utility>
 #include <vector>
 
@@ skipping 92 matching lines @@
     return card_info_container;
   }
 
   void SelectedStateChanged() override {
     if (selected()) {
       state()->SetSelectedInstrument(instrument_);
       dialog_->GoBack();
     }
   }
 
-  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();
   }
 
   void PerformSelectionFallback() override {
     switch (instrument_->type()) {
       case PaymentInstrument::Type::AUTOFILL:
         // Since we are a list item, we only care about the on_edited callback.
         dialog_->ShowCreditCardEditor(
@@ skipping 81 matching lines @@
                          base::Unretained(state()), /*selected=*/true),
           /*credit_card=*/nullptr);
       break;
     default:
       PaymentRequestSheetController::ButtonPressed(sender, event);
       break;
   }
 }
 
 }  // namespace payments