[MD Settings] Allow editing autofill items from Google Payments.

[MD Settings] Allow editing autofill items from Google Payments.

Also adds a "Google Payments" label to CCs and Addresses that come from Google Payments. Screenshot in Issue 607348.

BUG=607347, 607348
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/f0018ceaa583322e8732ef8bec460e82ff877918
Cr-Commit-Position: refs/heads/master@{#401473}

[hcarmona, 2016-06-21 01:59:54]

Description was changed from

==========
[MD Settings] Allow editing autofill items from Google Payments.

BUG=607347,607348
==========

to

==========
[MD Settings] Allow editing autofill items from Google Payments.

BUG=607347,607348
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[hcarmona, 2016-06-21 21:48:47]

Patchset #1 (id:1) has been deleted

[hcarmona, 2016-06-21 21:50:40]

Description was changed from

==========
[MD Settings] Allow editing autofill items from Google Payments.

BUG=607347,607348
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD Settings] Allow editing autofill items from Google Payments.

Also adds a "Google Payments" label to CCs and Addresses that come from Google
Payments. Screenshot in Issue 607348.

BUG=607347,607348
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[hcarmona, 2016-06-21 21:55:39]

hcarmona@chromium.org changed reviewers:
+ dschuyler@chromium.org

[hcarmona, 2016-06-21 21:56:12]

PTAL

[dschuyler, 2016-06-21 23:38:35]

lgtm

https://codereview.chromium.org/2083903002/diff/20001/chrome/app/settings_str...
File chrome/app/settings_strings.grdp (right):

https://codereview.chromium.org/2083903002/diff/20001/chrome/app/settings_str...
chrome/app/settings_strings.grdp:224: <message
name="IDS_SETTINGS_GOOGLE_PAYMENTS" desc="Label used to differentiate when an
address or credit card entry comes from Google Payments. This should follow the
casing of the 'Google Payments' brand.">
:) I like that you called out the casing of the brand.

[hcarmona, 2016-06-22 00:19:41]

Made a small change to CSS based on feedback on the screenshot.

Rebased branches which is why settings_strings.grdp is showing changes.

Still LG?

[dschuyler, 2016-06-22 23:00:36]

SLGTM

[hcarmona, 2016-06-22 23:59:05]

The CQ bit was checked by hcarmona@chromium.org

[hcarmona, 2016-06-22 23:59:05]

The patchset sent to the CQ was uploaded after l-g-t-m from
dschuyler@chromium.org
Link to the patchset: https://codereview.chromium.org/2083903002/#ps40001
(title: "fix margin and color")

[commit-bot: I haz the power, 2016-06-22 23:59:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2083903002/40001

[commit-bot: I haz the power, 2016-06-23 00:08:42]

Description was changed from

==========
[MD Settings] Allow editing autofill items from Google Payments.

Also adds a "Google Payments" label to CCs and Addresses that come from Google
Payments. Screenshot in Issue 607348.

BUG=607347,607348
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD Settings] Allow editing autofill items from Google Payments.

Also adds a "Google Payments" label to CCs and Addresses that come from Google
Payments. Screenshot in Issue 607348.

BUG=607347,607348
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-06-23 00:08:44]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-06-23 00:13:20]

Description was changed from

==========
[MD Settings] Allow editing autofill items from Google Payments.

Also adds a "Google Payments" label to CCs and Addresses that come from Google
Payments. Screenshot in Issue 607348.

BUG=607347,607348
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD Settings] Allow editing autofill items from Google Payments.

Also adds a "Google Payments" label to CCs and Addresses that come from Google
Payments. Screenshot in Issue 607348.

BUG=607347,607348
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/f0018ceaa583322e8732ef8bec460e82ff877918
Cr-Commit-Position: refs/heads/master@{#401473}
==========

[commit-bot: I haz the power, 2016-06-23 00:13:22]

Patchset 2 (id:??) landed as
https://crrev.com/f0018ceaa583322e8732ef8bec460e82ff877918
Cr-Commit-Position: refs/heads/master@{#401473}