1442757 - Forgot to call `sendMessageToContent` helper in "paymentDialogWrapper.js"

Status: RESOLVED FIXED

(Firefox :: WebPayments UI, defect, P1)

Description

[Jonathan Guillotte-Blouin [:jonathanGB]]

https://dxr.mozilla.org/mozilla-central/source/toolkit/components/payments/content/paymentDialogWrapper.js#261-266

This snippet should be replaced with the helper `sendMessageToContent` (https://dxr.mozilla.org/mozilla-central/source/toolkit/components/payments/content/paymentDialogWrapper.js#249).

Comment 1

[Jonathan Guillotte-Blouin [:jonathanGB]]

Attachment: Bug 1442757 - use `sendMessageToContent` helper.

Review commit: https://reviewboard.mozilla.org/r/224800/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/224800/

Comment 2

[Matthew N. [:MattN]]

Comment on attachment 8955726 [details]
Bug 1442757 - use `sendMessageToContent` helper.

https://reviewboard.mozilla.org/r/224800/#review230828

Thanks Jonathan!

::: commit-message-5a4ca:1
(Diff revision 1)
> +Bug 1442757 - use `sendMessageToContent` helper. r=mattn

Nit: in the future the commit message could be a bit more descriptive. e.g. adding "in `updateRequest`"

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4b3733f7e7f2
use `sendMessageToContent` helper. r=MattN

Comment 4

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/4b3733f7e7f2