Consider all monitors screen size when deciding to enable WebRender
Categories
(Core :: Graphics: WebRender, defect, P2)
Tracking
()
People
(Reporter: aosmond, Assigned: aosmond)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
lizzard
:
approval-mozilla-beta+
|
Details |
We currently disable WebRender when the screen size is too large for some configurations. We should actually take all monitors into consideration when making this decision, not just the primary monitor.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Before this patch, we only considered the primary screen when deciding
whether or not WebRender should be enabled. This is problematic for
Intel users where we don't want to turn on WebRender for large screens;
several small monitors are just as bad as one large screens. Now we add
up the pixel count for all the monitors when making this decision.
Comment 3•5 years ago
|
||
bugherder |
Comment 4•5 years ago
|
||
Is this needed on Beta for the Intel WebRender rollout? If so, Fx70 RC week is next week, so it'll need to be uplifted soon.
Assignee | ||
Comment 5•5 years ago
|
||
Comment on attachment 9093128 [details]
Bug 1581606 - Consider all screens when deciding to turn on WebRender.
Beta/Release Uplift Approval Request
- User impact if declined: May get WebRender when it may be too slow.
- Is this code covered by automated tests?: Yes
- 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): Makes users less likely to get WebRender, but only if they have multiple monitors with Intel graphics cards.
- String changes made/needed:
Comment on attachment 9093128 [details]
Bug 1581606 - Consider all screens when deciding to turn on WebRender.
Good idea. Let's uplift for beta 14.
Comment 7•5 years ago
|
||
bugherder uplift |
Description
•