Closed Bug 1265915 Opened 4 years ago Closed 4 years ago

Remove adaptive pings from the WebSocket backend

Categories

(Core :: DOM: Push Notifications, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
firefox49 --- fixed

People

(Reporter: Lina, Assigned: Lina)

References

Details

(Whiteboard: btpp-active)

Attachments

(1 file)

This is a complex bit of logic that isn't used at all on Desktop or Fennec. :mt favors server-driven pings for more exotic constrained platforms that we might want to support in the future.
See Also: → 1265914
Comment on attachment 8743059 [details]
MozReview Request: Bug 1265915 - Remove adaptive pings from the WebSocket backend. r?dragana

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/47539/diff/1-2/
Blocks: 1265914
See Also: 1265914
Comment on attachment 8743059 [details]
MozReview Request: Bug 1265915 - Remove adaptive pings from the WebSocket backend. r?dragana

https://reviewboard.mozilla.org/r/47539/#review45451

looks good.
Attachment #8743059 - Flags: review?(dd.mozilla)
Thanks, Dragana! Did you mean to r+ this and bug 1265915? :-)

https://treeherder.mozilla.org/#/jobs?repo=try&revision=02252e5dd398
Flags: needinfo?(dd.mozilla)
Oops, I meant bug 1265914.
Comment on attachment 8743059 [details]
MozReview Request: Bug 1265915 - Remove adaptive pings from the WebSocket backend. r?dragana

https://reviewboard.mozilla.org/r/47539/#review45565
Attachment #8743059 - Flags: review+
Thanks!
Flags: needinfo?(dd.mozilla)
https://hg.mozilla.org/mozilla-central/rev/f9aaa24d0f85
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Target Milestone: mozilla48 → mozilla49
You need to log in before you can comment on or make changes to this bug.