Video elements with flipped x-axis are turned upside down (user video on meet.jit.si and similar)
Categories
(Core :: Graphics: WebRender, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr91 | --- | unaffected |
firefox96 | --- | wontfix |
firefox97 | --- | fixed |
firefox98 | --- | fixed |
People
(Reporter: rmader, Assigned: rmader)
References
(Blocks 1 open bug, Regression)
Details
(Keywords: regression)
Attachments
(2 files, 1 obsolete file)
(deleted),
image/png
|
Details | |
(deleted),
text/x-phabricator-request
|
dmeehan
:
approval-mozilla-beta+
diannaS
:
approval-mozilla-release-
|
Details |
From FF 96 on, users of WebRTC services that show the users own video flipped (so it looks like a mirror) will see that video upside down.
While this might be a pure duplicate of bug 1750146, maybe it isn't and we're missing certain buffer rotation values.
Updated•3 years ago
|
Assignee | ||
Comment 1•3 years ago
|
||
This code never run before D132738. Turns out that the Wayland value
for flipped
exactly matches the WR one.
The wrong value was set as before D118925, WL_OUTPUT_TRANSFORM_FLIPPED_180
was used in various places.
Updated•3 years ago
|
Assignee | ||
Comment 2•3 years ago
|
||
This code never run before D132738. Turns out that the Wayland value
for flipped
exactly matches the WR one.
The wrong value was set as before D118925, WL_OUTPUT_TRANSFORM_FLIPPED_180
was used in various places.
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Pushed by robert.mader@posteo.de: https://hg.mozilla.org/integration/autoland/rev/7a9f266f68a5 Add support for x-flips in SetBufferTransformFlipped(), r=gfx-reviewers,mstange
Assignee | ||
Comment 4•3 years ago
|
||
Comment on attachment 9259231 [details]
Bug 1750373 - Add support for x-flips in SetBufferTransformFlipped(), r=#gfx-reviewers
Beta/Release Uplift Approval Request
- User impact if declined: There is a small group of people who run the experimental compositor backend as their daily drivers, especially Wayland devs (including me) and enthusiasts, in order to find remaining bugs in Firefox or, importantly, Wayland compositors.
The bug fixed here makes video conferences via WebRTC quite unpleasant and irritating as the users own video will be shown upside down. As video conferences are indispensable for many people right now, chances are high that these valuable people will turn off the feature if the issue is not fixed soon.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): The changes are completely contained in code that is only ever run when enabling the experimental backend. Also the code changes are very simple.
- String changes made/needed:
Assignee | ||
Comment 5•3 years ago
|
||
Comment on attachment 9259231 [details]
Bug 1750373 - Add support for x-flips in SetBufferTransformFlipped(), r=#gfx-reviewers
Beta/Release Uplift Approval Request
- User impact if declined: There is a small group of people who run the experimental compositor backend as their daily drivers, especially Wayland devs (including me) and enthusiasts, in order to find remaining bugs in Firefox or, importantly, Wayland compositors.
The bug fixed here makes video conferences via WebRTC quite unpleasant and irritating as the users own video will be shown upside down. As video conferences are indispensable for many people right now, chances are high that these valuable people will turn off the feature if the issue is not fixed soon. That could hamper further development.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): The changes are completely contained in code that is only ever run when enabling the experimental backend. Also the code changes are very simple.
- String changes made/needed:
Comment 6•3 years ago
|
||
bugherder |
Assignee | ||
Comment 7•3 years ago
|
||
Confirmed fix on nightly now, updated uplift requests.
Updated•3 years ago
|
Comment 8•3 years ago
|
||
Set release status flags based on info from the regressing bug 1744117
Comment 9•3 years ago
|
||
Comment on attachment 9259231 [details]
Bug 1750373 - Add support for x-flips in SetBufferTransformFlipped(), r=#gfx-reviewers
Approved for 97.0b5. Thanks.
Comment 10•3 years ago
|
||
bugherder uplift |
Updated•3 years ago
|
Updated•3 years ago
|
Description
•