1713002 - Hang when loading https://www.monbureaunumerique.fr/ on Fenix because of SVG filters

Status: NEW

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

Description

[Kevin Brosnan [Ex-Mozilla]]

From github: https://github.com/mozilla-mobile/fenix/issues/19678.

Steps to reproduce

• go to https://www.monbureaunumerique.fr/

Expected behavior

be able to navigate on the site as on firefox desktop

Actual behavior

firefox has a weird bug, the page is not displayed and cannot close the tab or change the url

Device information

• Device vendor / model and Android version: tested on Pixel XL android 9 and 10
• Firefox for Android version: 88.1.3

Change performed by the Move to Bugzilla add-on.

Comment 1

[Kevin Brosnan [Ex-Mozilla]]

Profile is https://share.firefox.dev/3fokTTB

Comment 2

[simon]

I can confirm this bug with Firefox Nightly 90.0a1 on my Google Pixel XL android 10
my phone is not very powerful it may be related to the bug

Comment 3

[simon]

The website works perfectly on a virtual device: Google Pixel 4XL, Android 11, Firefox Nightly 90.0a1

Comment 4

[AngelOfDarkness]

As I had written yesterday in the Github Issue Tracker:

I can confirm this issue only with my smartphone (Huawei P smart (2017, 3GB, Android 9.1.0) and the latest Play Store version.
But not complete as the opener has reported. The firsts moments while and after loading the tab Firefox didn't react.
After a few seconds and a previous click on a button Firefox react and the site will be displayed.

On my Chromebook with Intel i5 (8GB) and Android 11 Firefox 89.1.0 and nightly I don't have any problems with the reported site.

Comment 5

[Emily Toop (:fluffyemily)]

Aaron, can you help figure out where this should be redirected.

Comment 6

[(No longer employed by Mozilla) Aaron Klotz]

I see a 10.9s hang on the compositor thread.

Comment 7

[Sotaro Ikeda [:sotaro]]

(In reply to Kevin Brosnan [:kbrosnan] from comment #1)

Profile is https://share.firefox.dev/3fokTTB

WRWorker#7 was very busy by the following functions. In the functions, gfx::FilerNode functions were called. I wonder if it might be a regression by Bug 1704792.

• mozilla::gfx::InlineTranslator::TranslateRecording(char*, unsigned long)
• wr_moz2d_render_cb

Comment 8

[Sotaro Ikeda [:sotaro]]

:aosmond, can you comment to the bug?

Comment 9

[Robert Longson [:longsonr]]

does bug 1713350 improve things?

Comment 10

[Andrew Osmond [:aosmond] (he/him)]

The original issue was filed against release Firefox 88. It is possible I made it worse but it seems unlikely that was the original issue. Can we get a profile from 88 and/or 89, and a new profile from the latest 90?

I tested with WR (Z3C + LineageOS / Android 10 -- not particularly powerful) on the latest nightly, which includes the patch from bug 1713350, and it seems fine.

Comment 11

[Andrew Osmond [:aosmond] (he/him)]

Bug 1713651 turned off bug 1704792 in both nightly and beta so it should no longer be a factor in this performance issue.

Comment 12

[Kevin Brosnan [Ex-Mozilla]]

Using a Pixel 2XL on Android 11

• Geckoview example 91 - https://share.firefox.dev/3wK6Jli
• Firefox 90 - 8.2s hang mozilla::layers::WebRenderBridgeParent::CompositeToTarget https://share.firefox.dev/3yTkOik
• Firefox 89 - 8.3s hang mozilla::layers::WebRenderBridgeParent::CompositeToTarget  https://share.firefox.dev/3pcmtv0

Make sure that even when the throbber shows that the page is loaded that the images below the fold are loaded and the page scrolls smoothly.

Comment 13

[Jeff Muizelaar [:jrmuizel]]

This is happening because of SVG filters.

Comment 14

[Jeff Muizelaar [:jrmuizel]]

Here's the filter in question:

    <defs>
        <filter id="filter--blur" x="0" y="0">
            <feGaussianBlur in="SourceGraphic" stdDeviation="20"></feGaussianBlur>
            <feMorphology operator="dilate" radius="16"></feMorphology>
            <feMorphology operator="dilate" radius="6"></feMorphology>
        </filter>
    </defs>

We do not accelerate feMorphology. All that being said, it does seem like this is showing up as more expensive than it should be.