Closed
Bug 1279039
Opened 8 years ago
Closed 8 years ago
Pref off tmmbr
Categories
(Core :: WebRTC: Signaling, defect, P1)
Core
WebRTC: Signaling
Tracking
()
RESOLVED
FIXED
mozilla50
Tracking | Status | |
---|---|---|
firefox47 | --- | unaffected |
firefox48 | --- | unaffected |
firefox49 | --- | fixed |
firefox50 | --- | fixed |
People
(Reporter: mjf, Assigned: mjf)
Details
Attachments
(1 file)
58 bytes,
text/x-review-board-request
|
jesup
:
review+
lizzard
:
approval-mozilla-aurora+
|
Details |
If first TMMBR packets from Spark arrive with MaxTotalMediaBitRate > 300 we get a video freeze. IF the initial MaxTotalMediaBitRate is less than 300 it seems to work properly. For now, pref off.
Assignee | ||
Updated•8 years ago
|
status-firefox47:
--- → unaffected
status-firefox48:
--- → unaffected
status-firefox49:
--- → affected
status-firefox50:
--- → affected
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → mfroman
Assignee | ||
Comment 1•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/58586/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/58586/
Attachment #8761362 -
Flags: review?(rjesup)
Comment 2•8 years ago
|
||
Comment on attachment 8761362 [details] Bug 1279039 - pref off tmmbr for now, https://reviewboard.mozilla.org/r/58586/#review55564
Attachment #8761362 -
Flags: review?(rjesup) → review+
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Comment 3•8 years ago
|
||
has problems to apply: applying 78a13bba7aaf patching file modules/libpref/init/all.js Hunk #1 FAILED at 392 1 out of 1 hunks FAILED -- saving rejects to file modules/libpref/init/all.js.rej patch failed to apply abort: fix up the working directory and run hg transplant --continue can you take a look, thanks!
Flags: needinfo?(mfroman)
Keywords: checkin-needed
Pushed by rjesup@wgate.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/dbb7cd8510a8 pref off tmmbr for now, r=jesup
Assignee | ||
Comment 5•8 years ago
|
||
Looks like jesup took care of it already. Thanks!
Flags: needinfo?(mfroman)
Comment 6•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/dbb7cd8510a8
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
Assignee | ||
Comment 7•8 years ago
|
||
Comment on attachment 8761362 [details] Bug 1279039 - pref off tmmbr for now, Approval Request Comment [Feature/regressing bug #]: Bug 1270230 pref'd on, pref-ing off here. [User impact if declined]: Video freezes when TMMBR packets arrive with MaxTotalMediaBitRate > 300 using Cisco Spark. [Describe test coverage new/current, TreeHerder]: On inspection when using Spark especially over a long-distance VPN connection to provoke the issue. [Risks and why]: Very little risk in turning this pref back off since it was off for quite some time prior to turning it on in 1270230. [String/UUID change made/needed]: none
Attachment #8761362 -
Flags: approval-mozilla-aurora?
Comment 8•8 years ago
|
||
Comment on attachment 8761362 [details] Bug 1279039 - pref off tmmbr for now, Keeping this pref off in 49 as video freezes aren't fun.
Attachment #8761362 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in
before you can comment on or make changes to this bug.
Description
•