1149861 - Implement getMaxTouchPoints for Android

Status: RESOLVED FIXED

(Core :: Widget: Android, defect)

Description

[Makoto Kato [:m_kato]]

Although we disable dom.w3c_pointer_events.enabled by default, Chrome for Android already has this method.

Comment 1

[Makoto Kato [:m_kato]]

Attachment: add GetMaxTouchPoints

Comment 2

[James Willcox (:snorp) ([e-mail]) (he/him)]

Comment on attachment 8586558 [details] [diff] [review]
add GetMaxTouchPoints

Review of attachment 8586558 [details] [diff] [review]:
-----------------------------------------------------------------

I would love it if we could avoid adding another method to GeckoAppShell for this, but I guess it's fine for now.

::: mobile/android/base/GeckoAppShell.java
@@ +2678,5 @@
> +
> +    @WrapElementForJNI
> +    static int getMaxTouchPoints() {
> +        PackageManager pm = getContext().getPackageManager();
> +        if (pm.hasSystemFeature(PackageManager.FEATURE_TOUCHSCREEN_MULTITOUCH_JAZZHAND)) {

lol

Comment 3

[Maksim Lebedev]

We should wait any changes for this?

Comment 4

[Makoto Kato [:m_kato]]

(In reply to Maksim Lebedev from comment #3)
> We should wait any changes for this?

Ah, I forget landing this. sorry

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/91f9679c4665

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/91f9679c4665