change some string types in wallet code

change some string types in wallet code

BUG=none


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=183429

[Evan Stade, 2013-02-13 23:34:59]

strings that are used for display should be string16. I tried to only convert
strings that are primarily used for display, but I'm not sure I accomplished
this, since most of these member variables aren't actually used for anything at
the moment, and the documentation is somewhat lacking.

Autofill also uses string16 to store values that will be filled into web forms,
so to follow that convention, strings that are used for that purpose as well as
display can/should be string16.

[Ilya Sherman, 2013-02-14 00:03:13]

lgtm

[ahutter, 2013-02-14 22:17:01]

lgtm

https://codereview.chromium.org/12262028/diff/1/chrome/browser/autofill/walle...
File chrome/browser/autofill/wallet/instrument.h (right):

https://codereview.chromium.org/12262028/diff/1/chrome/browser/autofill/walle...
chrome/browser/autofill/wallet/instrument.h:67: std::string
primary_account_number_;
This is user input.

https://codereview.chromium.org/12262028/diff/1/chrome/browser/autofill/walle...
chrome/browser/autofill/wallet/instrument.h:70: std::string
card_verification_number_;
ditto.

[ahutter, 2013-02-14 22:17:02]

lgtm

[commit-bot: I haz the power, 2013-02-19 17:53:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/[email protected]/12262028/6001

[commit-bot: I haz the power, 2013-02-20 04:14:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/[email protected]/12262028/18001

[commit-bot: I haz the power, 2013-02-20 08:11:20]

Change committed as 183429