[Payment Request] Updates iOS watchlist

[Payment Request] Updates iOS watchlist

Review-Url: https://codereview.chromium.org/2709973003
Cr-Commit-Position: refs/heads/master@{#452158}
Committed: https://chromium.googlesource.com/chromium/src/+/0a7bb59fff7ca4ba57577a4dcf3e013019f5794b

[Moe, 2017-02-22 14:53:58]

mahmadi@chromium.org changed reviewers:
+ rouslan@chromium.org

[Moe, 2017-02-22 14:53:59]

Hi Rouslan. PTAL.

[please use gerrit instead, 2017-02-22 15:07:14]

https://codereview.chromium.org/2709973003/diff/1/WATCHLISTS
File WATCHLISTS (right):

https://codereview.chromium.org/2709973003/diff/1/WATCHLISTS#newcode822
WATCHLISTS:822: '|ios/web/public/payments/'\
Does iOS use components/payments/ ?

[Moe, 2017-02-22 15:12:26]

https://codereview.chromium.org/2709973003/diff/1/WATCHLISTS
File WATCHLISTS (right):

https://codereview.chromium.org/2709973003/diff/1/WATCHLISTS#newcode822
WATCHLISTS:822: '|ios/web/public/payments/'\
On 2017/02/22 15:07:13, rouslan wrote:
> Does iOS use components/payments/ ?

It does. I think people should subscribe to 'payments' to keep up with the
payments code. This is just to keep up with iOS specific CLs.

[please use gerrit instead, 2017-02-22 15:15:57]

LGTM % comment: I recommend adding "components/payments/currency_formatter" to
the payments_ios watchlist. Since iOS uses that include file, any changes to it
will affect you directly, so you want to be aware of any upcoming changes in
there.

[Moe, 2017-02-22 18:47:34]

Thanks Rouslan, I added myself to the 'payments' watchlist as well to keep
up-to-date wit the components/payments changes incl. the CurrencyFormatter.

[Moe, 2017-02-22 18:47:38]

The CQ bit was checked by mahmadi@chromium.org

[Moe, 2017-02-22 18:47:38]

The patchset sent to the CQ was uploaded after l-g-t-m from rouslan@chromium.org
Link to the patchset: https://codereview.chromium.org/2709973003/#ps20001
(title: "Added self to 'payments' watch list")

[commit-bot: I haz the power, 2017-02-22 18:50:16]

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-02-22 19:13:54]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487789258118270,
"parent_rev": "23b7d550c24b548b910cc85fd2ac30916f22e3a8", "commit_rev":
"0a7bb59fff7ca4ba57577a4dcf3e013019f5794b"}

[commit-bot: I haz the power, 2017-02-22 19:14:40]

Description was changed from

==========
[Payment Request] Updates iOS watchlist
==========

to

==========
[Payment Request] Updates iOS watchlist

Review-Url: https://codereview.chromium.org/2709973003
Cr-Commit-Position: refs/heads/master@{#452158}
Committed:
https://chromium.googlesource.com/chromium/src/+/0a7bb59fff7ca4ba57577a4dcf3e...
==========

[commit-bot: I haz the power, 2017-02-22 19:14:41]

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