[Payments] Selecting incomplete items will open editors

components/payments/content/payment_request_state.cc

Index: components/payments/content/payment_request_state.cc
diff --git a/components/payments/content/payment_request_state.cc b/components/payments/content/payment_request_state.cc
index 0a1a487b5aaf6c6f85399733044507f0f384f124..4a2dc5a94e82aa8d1bf6538b91da184168d2c1d7 100644
--- a/components/payments/content/payment_request_state.cc
+++ b/components/payments/content/payment_request_state.cc
@@ -41,8 +41,7 @@ PaymentRequestState::~PaymentRequestState() {}
 bool PaymentRequestState::CanMakePayment() const {
   for (const std::unique_ptr<PaymentInstrument>& instrument :
        available_instruments_) {
-    if (instrument.get()->IsValid() &&

[anthonyvd, 2017/04/07 20:08:09]

Is this just that the PR spec doesn't require the instrument to be valid for
CanMakePayment to return true?

[Mathieu, 2017/04/09 00:35:45]

You're right this was unclear. I've rearranged so that the PaymentInstrument API
is clearer, and implemented it to be the same as in Java.

-        spec_->supported_card_networks_set().count(
+    if (spec_->supported_card_networks_set().count(
             instrument.get()->method_name())) {
       return true;
     }