Re: [PATCH v5 0/1] MR5861: mshtml: Implement visibilitychange event.
On Wed Dec 3 15:43:01 2025 +0000, Jacek Caban wrote:
Note that minimizing a window is just one case where this event should fire, but there are many others. In regular browsers, switching tabs is probably the most common source of the event. To support that, we would need to detect real visibility changes of the document, not just window minimization. That said, we do not need a complete solution right away, but since even the limited support already requires awkward hooks, this does not look very appealing. What exactly is it fixing? Do you have an actual application that needs it? And what is the observable behavior difference between this MR and just a stub? For example some games/apps XBox Sign-In is using it (via the Microsoft libs), probably for some sort of telemetry since it also does a bunch of performance timings, though I've no idea of the exact impact vs a stub. I actually wanted to have the tests not skipped on Wine (and also because we didn't have the hook before Esme implemented it, and now wanted to try it out).
I mean, even if we do include it in other sources like switching tabs (although I'm unsure how to do that from mshtml without IE? aren't tabs in iexplore instead?), we'd still have to handle minimization somehow. And at least that one is more important since it works 100% without iexplore. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5861#note_124564
participants (1)
-
Gabriel Ivăncescu