1710708 - Overscroll is not displayed on Android 10 and later

Status: RESOLVED DUPLICATE of bug 1707959

(GeckoView :: General, defect, P3)

Description

[[ex-Mozilla] Agi Sferro | :agi]

Not sure if this is a bug for earlier Android versions, but on 10 and later overscroll is not displayed. From local debugging it appears that this https://searchfox.org/mozilla-central/rev/6b099d836c882bc155d2ef285e0ad0ab9f5038f6/mobile/android/geckoview/src/main/java/org/mozilla/geckoview/OverscrollEdgeEffect.java#62 is throwing and being silently ignored which might be the cause of this.

Comment 1

[[ex-Mozilla] Agi Sferro | :agi]

Reflection is likely failing because of the restriction added in Android 9: https://developer.android.com/guide/app-compatibility/restrictions-non-sdk-interfaces

There is a new API now, EdgeEffect.setBlendMode https://developer.android.com/reference/android/widget/EdgeEffect#setBlendMode(android.graphics.BlendMode) which looks like could be used in place of reflection on newer Android versions.

Comment 2

[Emily Toop (:fluffyemily)]

Emily to talk to Vesta about priorities for this work

Comment 4

[Ilya K]

This (or rather the situation described in bug 1738691) happens a lot more often for me on Android 12 :(

Comment 5

[Makoto Kato [:m_kato]]

I think that https://github.com/mozilla-mobile/fenix/issues/25592 and https://github.com/mozilla-mobile/fenix/issues/22234 are another issue. No method in AndroidContentController whether pan is end or touch is released.

This is rendering issue of overscroll. Why does not EdgeEffect.draw draws overscroll Semicircle...

Comment 6

[Makoto Kato [:m_kato]]

My Android 11 device and Android 11 emulator image draw overscoll animation by bug 1707959 when disabling GPU process (bug 1785786).

Also, Android 12+ (Target API 31+) will use stretch animation instead of glow drawing.

Comment 7

[Makoto Kato [:m_kato]]

After landing bug 1785786, Android 10 (Emulator image) and Android 11 (Xiaomi Redmi 9T) works glow drawing animation for overscoll. By bug 1707959, this is fixed.

Also, I filed new issue for Android 12+ as bug 1786452.