Closed Bug 1809519 Opened 1 year ago Closed 1 year ago

Enable the GPU sandbox in Release

Categories

(Core :: Widget: Win32, enhancement, P1)

All
Windows
enhancement

Tracking

()

RESOLVED FIXED
110 Branch
Tracking Status
relnote-firefox --- 110+
firefox108 --- wontfix
firefox109 --- wontfix
firefox110 --- fixed

People

(Reporter: cmartin, Assigned: cmartin)

References

Details

Attachments

(1 file)

The GPU sandbox has been enabled in Beta for a while, and all has been pretty quiet. Let's flip the switch and send it to release!

Assignee: nobody → cmartin
Severity: -- → S3
Status: NEW → ASSIGNED
Priority: -- → P1

I am guessing that this will land in 110 before the soft code freeze, will that require QA? An addition to release notes? Thanks

Flags: needinfo?(cmartin)

Note that we are going to enable this in Release channel despite the fact that Bug 1799470 is unresolved. Though it may not be as strong of a sandbox, a weakened GPU sandbox is still an improvement over no sandbox at all. A fix for that bug is likely coming in the next release or two anyhow. (famous last words...)

(In reply to Pascal Chevrel:pascalc from comment #2)

I am guessing that this will land in 110 before the soft code freeze, will that require QA? An addition to release notes? Thanks

Yes, we are going to try and get it enabled in 110. I don't think that it will require any special QA, as a lot of this is going to end up being corner cases that we're going to run into on individual users' machine configurations, and I wouldn't even know what to ask of QA in addition to the usual testing.

Good call on the Release Notes though -- Definitely need to add that.

Release Note Request (optional, but appreciated)
[Why is this notable]: It might cause glitchy behavior, performance issues, or other difficult-to-anticipate behavior on user's machines
[Affects Firefox for Android]: No
[Suggested wording]: "GPU sandboxing has been enabled on Windows"
[Links (documentation, blog post, etc)]:

relnote-firefox: --- → ?
Flags: needinfo?(cmartin)
Pushed by cmartin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/381ff7fccb6f
Enable the GPU sandbox in Release r=jrmuizel
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 110 Branch

Note added to our Nightly and Beta notes, thanks.

Hi Pascal,

Could you please also add an additional note to the end of the previous release note for this:

Note: A bug in the popular X-Mouse Button Control (XMBC) tool may cause mouse wheel scrolling to stop working. The author(s) are working on an update.

Thanks!

Flags: needinfo?(pascalc)

(In reply to Chris Martin [:cmartin] from comment #9)

Hi Pascal,

Could you please also add an additional note to the end of the previous release note for this:

Note: A bug in the popular X-Mouse Button Control (XMBC) tool may cause mouse wheel scrolling to stop working. The author(s) are working on an update.

Thanks!

Note added.

Flags: needinfo?(pascalc)

Could we also mention explicitly the existing fix in the note? We could add a link to bug 1798014 comment 78, or copy paste some of its content in the note; maybe the second option is more appropriate.

Flags: needinfo?(pascalc)

(In reply to Yannis Juglaret from comment #11)

Could we also mention explicitly the existing fix in the note? We could add a link to bug 1798014 comment 78, or copy paste some of its content in the note; maybe the second option is more appropriate.

Please provide the exact wording you would like to have for the note, thanks.

Flags: needinfo?(pascalc)
Flags: needinfo?(yjuglaret)

Sure, sorry about that. I guess something like this, if Chris agrees?

Note: A bug in the popular X-Mouse Button Control (XMBC) tool may cause mouse wheel scrolling to stop working. The author(s) are working on an update. Meanwhile, scrolling can be restored by reconfiguring XMBC: either disable the Make scroll wheel scroll window under cursor option in the global settings, or enable the Disable scroll window under cursor option if using a custom profile for Firefox.

Flags: needinfo?(yjuglaret) → needinfo?(cmartin)

Thanks for bringing this up, Yannis, and for providing the wording -- Looks good to me! :)

Flags: needinfo?(cmartin)

Note updated, will be live in 30mn.

Added to final 110 notes.

See Also: → 1823412
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: