1766094 - SVG animation not rendered while page is idle

Status: ASSIGNED

(Core :: Graphics: WebRender, defect, P3)

Description

[q3.max.2011]

Attachment: splash.svg

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:101.0) Gecko/20100101 Firefox/101.0

Steps to reproduce:

Open any page containing SVG image with animated HTML component inside and let page sit idle. (example https://github.com/maxsupermanhd/WebChunk/blob/master/splash.svg)

Prior to some recent nightly update it was working fine.

Example SVG in attachment.

Firefox Nightly 101.0a1 (2022-04-22) (64-bit) on XFCE 5.13.0-35-generic #40~20.04.1-Ubuntu (NVIDIA proprietary driver 510)

Actual results:

Animation is not being rendered (looks like it is being processed but tab does not show any visual changes) if inside HTML page. Renders as usual if opened just as image file by itself.

Expected results:

Animated SVG components should be always rendered regardless of tab being idle.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Core::Graphics: WebRender' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Comment 2

[Timothy Nikkel (:tnikkel)]

Seems to work fine on mac. Would you be willing to use https://mozilla.github.io/mozregression/ to find out what caused this?

Comment 3

[q3.max.2011]

I tested it with mozregression GUI and attached screenshot after final test: https://imgur.com/a/ouiSXNW

It seems like it is related to bug 1763643 and 1763642, build 4a5f91ff does not work correctly but 3d61c0e3 renders continuous animation just fine.

Comment 4

[Timothy Nikkel (:tnikkel)]

Thanks!

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1763643

Comment 6

[BugBot [:suhaib / :marco/ :calixte]]

:aosmond, since you are the author of the regressor, bug 1763643, could you take a look?
For more information, please visit auto_nag documentation.

Comment 7

[Jamie Nicol [:jnicol]]

The regressor is being backed out in bug 1766333

Comment 8

[Ryan VanderMeulen [:RyanVM]]

Disabled by way of bug 1766333.

Comment 9

[BugBot [:suhaib / :marco/ :calixte]]

The bug has a release status flag that shows some version of Firefox is affected, thus it will be considered confirmed.

Comment 10

[Timothy Nikkel (:tnikkel)]

Attachment: Bug 1766094. If we received an UpdateBlobImage then mark the frame invalid so we update it. r?#gfx-reviewers

All other UpdateBlobImage updates come during a transaction with a new display list, so they don't hit this path. However svg blob images can update without a new display list, by just sending a resource update. This matches how the same code for animated raster images uses UpdateImage.

Comment 11

[BugBot [:suhaib / :marco/ :calixte]]

There is an r+ patch which didn't land and no activity in this bug for 2 weeks.
:tnikkel, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit BugBot documentation.