1066237 - MaxTouchPoints variable may return invalid value on the win7 platform

Status: RESOLVED FIXED

(Core :: DOM: Events, defect)

Description

[Nick Lebedev [:nl]]

Please read the following thread for more info:
http://lists.w3.org/Archives/Public/public-pointer-events/2014JulSep/0116.html

Comment 1

[Olli Pettay [:smaug]]

http://mxr.mozilla.org/mozilla-central/source/dom/webidl/Navigator.webidl#146

It doesn't

Comment 2

[Olli Pettay [:smaug]]

Though, I guess we want to change the implementation anyhow.

Comment 3

[Nick Lebedev [:nl]]

Yes, we might need to add the same check for 'IsTouchDevicePresent' as Chrome/Blink guys did. Just for future days when pointer events preference is enabled by default.

Comment 4

[Nick Lebedev [:nl]]

Attachment: Add explicit returning zero number of touch points when there is no touch device available.

Olli,

Blink guys said on the pointer events mailing list that sometimes implementation we had for maxTouchPoints by some reason return "1" instead of "0" when there is no touch screen at all - http://lists.w3.org/Archives/Public/public-pointer-events/2014JulSep/0110.html.

And to address it they landed the fix: http://lists.w3.org/Archives/Public/public-pointer-events/2014JulSep/0127.html.

Possibly we need to do the similar thing to prevent getting the same bug that Chrome got when the pointer events get turned on by default in Firefox. Could you take a look at the mails and patch and let me know if we should do it.

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/7d6eb07eb884

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/7d6eb07eb884