Closed Bug 1088230 Opened 6 years ago Closed 6 years ago

Add backwards compatible attribute to LoopServer registration payload

Categories

(Hello (Loop) :: Client, defect)

defect
Not set
normal
Points:
1

Tracking

(firefox35 fixed, firefox36 fixed)

RESOLVED FIXED
mozilla36
Iteration:
36.1
Tracking Status
firefox35 --- fixed
firefox36 --- fixed

People

(Reporter: pkerr, Assigned: pkerr)

References

Details

(Whiteboard: [rooms])

Attachments

(1 file, 1 obsolete file)

Add the simplePushURL attribute to the registration payload along with the simplePushURLs attribute used by the new rooms server API. Assign simplePushURL the pushURL that is assigned to the calls client code.
Status: NEW → ASSIGNED
Attachment #8510528 - Flags: review?(standard8)
Comment on attachment 8510528 [details] [diff] [review]
add backwards compatible registration payload to new rooms registration

Review of attachment 8510528 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/components/loop/MozLoopService.jsm
@@ +518,5 @@
>    registerWithLoopServer: function(sessionType, pushUrls, retry = true) {
> +    // create a registration payload with a backwards compatible attribute (simplePushURL)
> +    // that will register only the calls notification.
> +    let msg = { simplePushURL: pushUrls.calls,
> +                simplePushURLs: pushUrls};

Please following the coding style with { and } ending and starting a new line respectively:
https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style#JavaScript_objects
Comment on attachment 8510528 [details] [diff] [review]
add backwards compatible registration payload to new rooms registration

Looks good, r=Standard with the change Matt suggested.
Attachment #8510528 - Flags: review?(standard8) → review+
Attachment #8510528 - Attachment is obsolete: true
Carrying forward r+ Standard8.
Keywords: checkin-needed
https://hg.mozilla.org/integration/fx-team/rev/fa426ab14a9c
Iteration: --- → 36.1
Keywords: checkin-needed
Target Milestone: --- → mozilla36
https://hg.mozilla.org/mozilla-central/rev/fa426ab14a9c
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Flags: qe-verify-
Flags: in-testsuite?
Comment on attachment 8510624 [details] [diff] [review]
add backwards compatible registration payload to new rooms registration

Approval Request Comment
Landed on aurora per IRC with lsblakk with a=loop-only
Attachment #8510624 - Flags: review+
Attachment #8510624 - Flags: approval-mozilla-aurora?
Attachment #8510624 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
This was a temporary change and has since been removed. Nothing to do on the testsuite side now.
Flags: in-testsuite?
You need to log in before you can comment on or make changes to this bug.