1692405 - [Bug] Pull to refresh / dynamic toolbars triggered on about:blank

Status: RESOLVED FIXED

(GeckoView :: General, defect, P2)

Description

[Petru-Mugurel Lingurar [:petru]]

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

Steps to reproduce

• Load about:blank (empty page, doesn't display anything)
• Swipe up and down

Expected behavior

Nothing would happen as the page is empty

Actual behavior

• pull to refresh throbber is shown and never dismisses since the triggered refresh never completes.
• the toolbars are scrolling even though no page content was scrolled.

Change performed by the Move to Bugzilla add-on.

Comment 1

[Petru-Mugurel Lingurar [:petru]]

I see the onTouchEventForResult() returning 1 (INPUT_RESULT_HANDLED) when swiping on about:blank based on which on Fenix the toolbars / pull to refresh throbber will be animated.
I think in this case a more appropriate return would be 0 (INPUT_RESULT_UNHANDLED).

Comment 2

[Emily Toop (:fluffyemily)]

Emily to ping Vesta to ask about Fenix priority.

Comment 3

[Petru-Mugurel Lingurar [:petru]]

This is not reproducible anymore. Possibly after https://bugzilla.mozilla.org/show_bug.cgi?id=1678505.

Comment 4

[Hiroyuki Ikezoe (:hiro)]

Thank you Petru for the confirmation!

Comment 5

[Chris Peterson [:cpeterson]]

Moving toolbar bugs to the new GeckoView::Toolbar component.