|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
Comment on attachment 8814178 [details] Bug 1320150: only fire consent when previously connected. https://reviewboard.mozilla.org/r/95440/#review96060 Looks good!
Attachment #8814178 - Flags: review?(mfroman) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/405a08f9905b only fire consent when previously connected. r=mjf
status-firefox50: --- → unaffected
status-firefox51: --- → unaffected
status-firefox52: --- → affected
status-firefox53: --- → affected
Comment on attachment 8814178 [details] Bug 1320150: only fire consent when previously connected. Approval Request Comment [Feature/Bug causing the regression]: Landing of bug 852665 in 52 caused this. [User impact if declined]: Some WebRTC services could assume the connection is already established even though not everything is connected yet. [Is this code covered by automated tests?] Yes bug 1316200 has mochitest work which verifies this. [Has the fix been verified in Nightly?] I have a mochitests patch over in bug 1316200 which tripped over this and thus verifies the correct/expected behavior. [Needs manual test from QE? If yes, steps to reproduce]: No. [List of other uplifts needed for the feature/fix]: None [Is the change risky?]: No. [Why is the change risky/not risky?]: This patch only adds another safe guard so the signal to JS from patch 852665 doesn't fire too early. [String changes made/needed]: N/A
Attachment #8814178 - Flags: approval-mozilla-aurora?
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox53: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Comment on attachment 8814178 [details] Bug 1320150: only fire consent when previously connected. fix a new webrtc bug in aurora52
Attachment #8814178 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
status-firefox52: affected → fixed
You need to log in before you can comment on or make changes to this bug.