Closed Bug 1567951 Opened 3 years ago Closed 3 years ago

Add implicit rollback to setRemoteDescription(offer) per spec.

Categories

(Core :: WebRTC: Signaling, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: jib, Assigned: jib)

References

()

Details

(Keywords: dev-doc-complete)

Attachments

(2 files)

Pushed by jbruaroey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/092e24332c39
Add implicit rollback in setRemoteDescription(offer). r=bwc
https://hg.mozilla.org/integration/autoland/rev/ca95add81105
Test implicit rollback in SRD(offer). r=bwc
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/18010 for changes under testing/web-platform/tests
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Flags: needinfo?(jib)
Upstream PR was closed without merging
Flags: needinfo?(jib)
Pushed by jbruaroey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3b1d51cda77c
Add implicit rollback in setRemoteDescription(offer). r=bwc
https://hg.mozilla.org/integration/autoland/rev/84386d8e67af
Test implicit rollback in SRD(offer). r=bwc
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70

For documentation purposes, we need to tweak some of the discussion and possibly sample code around negotiation slightly here.

Keywords: dev-doc-needed

Documentation updates:

  • Text updates to setRemoteDescription() are completed
  • Listed on Firefox 70 for developers
  • BCD PR 5018 submitted to update compatibility database to include an entry for "implicit_rollback".

The sample code and general tutorial text updates that need to be made for perfect negotiation will all happen at once after all of the required changes have been implemented. See the perfect negotiation doc updates issue on the doc team issue tracker for details.

You need to log in before you can comment on or make changes to this bug.