Scan card holder name.

components/autofill/core/browser/credit_card.h

Index: components/autofill/core/browser/credit_card.h
diff --git a/components/autofill/core/browser/credit_card.h b/components/autofill/core/browser/credit_card.h
index db9712ab9b78b14f824fd589fe8bedebcedbe9d0..f2ded811f564a16f7abe0f2cc357c52d5b9e5c53 100644
--- a/components/autofill/core/browser/credit_card.h
+++ b/components/autofill/core/browser/credit_card.h
@@ -46,7 +46,8 @@ class CreditCard : public AutofillDataModel {
   };
 
   CreditCard(const std::string& guid, const std::string& origin);
-  CreditCard(const base::string16& card_number,
+  CreditCard(const base::string16& card_holder_name,

[Evan Stade, 2016/08/16 00:33:35]

I don't see why we need a special constructor for this one use case

[please use gerrit instead, 2016/08/16 22:03:19]

Leaving the original constructor alone.

[Evan Stade, 2016/08/17 06:30:57]

cool, but I was actually hoping you'd delete it completely. It only appears to
be used in one place in production code, which is the card scanner. I don't
really think we should be pushing such specific (one off) uses into
constructors.

[please use gerrit instead, 2016/08/17 18:16:17]

This constructor is used in tons of tests, so it would be better to delete it in
a separate CL.

+             const base::string16& card_number,
              int expiration_month,
              int expiration_year);