1674982 - [Wayland] Revert Bug 1668771 (frame callbacks timeout) due to OOM crashes

Status: RESOLVED FIXED

(Core :: Widget: Gtk, defect, P3)

Description

[Martin Stránský [:stransky] (ni? me)]

We need to revert Bug 1668771 (frame callbacks timeout) due to OOM crashes.

Wayland protocol does not include a mechanism how to inform compositor about deleted frame callbacks. We delete the callback only locally on client side and it remains active at server (compositor) side.

When large amount of callbacks is accumulated at server Firefox is closed due to OOM.

Comment 1

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1674982 [Wayland] Revert frame callbacks timeout due to OOM crashes, r?jhorak

Wayland protocol does not include a mechanism how to inform compositor about deleted frame callbacks.
We delete a callback only locally on client side and it remains active at server (compositor) side.

When large amount of callbacks is accumulated at server Firefox is closed due to OOM
so we need to keep frame callback on local side untill is fired and don't create a new one.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3da7cf56aeb5
[Wayland] Revert frame callbacks timeout due to OOM crashes, r=jhorak

Comment 3

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/3da7cf56aeb5

Comment 4

[Pascal Chevrel:pascalc]

Martin, it seems we want this patch uplifted to 83 beta, right?

Comment 5

[Martin Stránský [:stransky] (ni? me)]

Yes, I'd like to uplift this one.
Thanks.

Comment 6

[Martin Stránský [:stransky] (ni? me)]

Comment on attachment 9185413 [details]
Bug 1674982 [Wayland] Revert frame callbacks timeout due to OOM crashes, r?jhorak

Beta/Release Uplift Approval Request

• User impact if declined: Firefox crashes when it's rendering web content on background, like youtube playback on background or so.
• 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): Low risk - it only reverts a single patch which causes the failure.
• String changes made/needed: none

Comment 7

[Pascal Chevrel:pascalc]

Comment on attachment 9185413 [details]
Bug 1674982 [Wayland] Revert frame callbacks timeout due to OOM crashes, r?jhorak

Linux recent crash fix on Wayland, approved for beta 9, thanks.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/d8118a49c5a8