Add Bank Name in sync protocol

Add Bank Name in sync protocol, to support https://docs.google.com/document/d/1muZH5IHNih1qj7xlICMxag2C1bklxyQnJ2gAsi6ld_4/edit

BUG=685742
Review-Url: https://codereview.chromium.org/2649873008
Cr-Commit-Position: refs/heads/master@{#446752}
Committed: https://chromium.googlesource.com/chromium/src/+/66660640fd0ed0b483330345072245a8d5a83a06

[Shanfeng, 2017-01-26 19:39:20]

Description was changed from

==========
Add Bank Name in sync protocol

BUG=
==========

to

==========
Add Bank Name in sync protocol, to support
https://docs.google.com/document/d/1muZH5IHNih1qj7xlICMxag2C1bklxyQnJ2gAsi6ld...

BUG=
==========

[Shanfeng, 2017-01-26 19:39:20]

szhangcs@google.com changed reviewers:
+ jsaul@google.com, zea@chromium.org

[Shanfeng, 2017-01-26 19:53:22]

szhangcs@google.com changed reviewers:
+ mathp@chromium.org

[Shanfeng, 2017-01-26 20:10:57]

Description was changed from

==========
Add Bank Name in sync protocol, to support
https://docs.google.com/document/d/1muZH5IHNih1qj7xlICMxag2C1bklxyQnJ2gAsi6ld...

BUG=
==========

to

==========
Add Bank Name in sync protocol, to support
https://docs.google.com/document/d/1muZH5IHNih1qj7xlICMxag2C1bklxyQnJ2gAsi6ld...

BUG=685742
==========

[Jared Saul, 2017-01-26 20:31:23]

https://codereview.chromium.org/2649873008/diff/1/components/sync/protocol/au...
File components/sync/protocol/autofill_specifics.proto (right):

https://codereview.chromium.org/2649873008/diff/1/components/sync/protocol/au...
components/sync/protocol/autofill_specifics.proto:127: // Bank name
I'd recommend being a little more descriptive, like maybe:

  // Issuing bank name (e.g. Chase)

[Shanfeng, 2017-01-26 21:25:02]

Thanks for review

https://codereview.chromium.org/2649873008/diff/1/components/sync/protocol/au...
File components/sync/protocol/autofill_specifics.proto (right):

https://codereview.chromium.org/2649873008/diff/1/components/sync/protocol/au...
components/sync/protocol/autofill_specifics.proto:127: // Bank name
On 2017/01/26 20:31:23, Jared Saul wrote:
> I'd recommend being a little more descriptive, like maybe:
> 
>   // Issuing bank name (e.g. Chase)

Done.

[Jared Saul, 2017-01-26 21:52:50]

lgtm

[Mathieu, 2017-01-26 22:05:01]

lgtm for this small change. Are you documenting all the changes to be made
somewhere?

[Nicolas Zea, 2017-01-27 00:41:00]

lgtm

[Shanfeng, 2017-01-27 18:09:27]

On 2017/01/26 22:05:01, Mathieu Perreault wrote:
> lgtm for this small change. Are you documenting all the changes to be made
> somewhere?

Here's a draft of my design doc: go/ib-bank-name

[Shanfeng, 2017-01-27 18:54:18]

The CQ bit was checked by szhangcs@google.com

[commit-bot: I haz the power, 2017-01-27 18:54:47]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-27 20:34:19]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485543258502500,
"parent_rev": "95e575c95dbae022874c992ac7c87cde406d38a3", "commit_rev":
"66660640fd0ed0b483330345072245a8d5a83a06"}

[commit-bot: I haz the power, 2017-01-27 20:34:52]

Description was changed from

==========
Add Bank Name in sync protocol, to support
https://docs.google.com/document/d/1muZH5IHNih1qj7xlICMxag2C1bklxyQnJ2gAsi6ld...

BUG=685742
==========

to

==========
Add Bank Name in sync protocol, to support
https://docs.google.com/document/d/1muZH5IHNih1qj7xlICMxag2C1bklxyQnJ2gAsi6ld...

BUG=685742

Review-Url: https://codereview.chromium.org/2649873008
Cr-Commit-Position: refs/heads/master@{#446752}
Committed:
https://chromium.googlesource.com/chromium/src/+/66660640fd0ed0b4833303450722...
==========

[commit-bot: I haz the power, 2017-01-27 20:34:53]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/66660640fd0ed0b4833303450722...