Closed Bug 1066237 Opened 5 years ago Closed 5 years ago

MaxTouchPoints variable may return invalid value on the win7 platform

Categories

(Core :: DOM: Events, defect)

x86_64
Windows 7
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla35

People

(Reporter: nl, Assigned: nl)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Please read the following thread for more info:
http://lists.w3.org/Archives/Public/public-pointer-events/2014JulSep/0116.html
Blocks: 996493
Blocks: pointerevent
No longer blocks: 996493
http://mxr.mozilla.org/mozilla-central/source/dom/webidl/Navigator.webidl#146

It doesn't
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → INVALID
Though, I guess we want to change the implementation anyhow.
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
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.
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.
Assignee: nobody → nicklebedev37
Attachment #8503222 - Flags: review?(bugs)
Attachment #8503222 - Flags: review?(bugs) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/7d6eb07eb884
Status: REOPENED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
You need to log in before you can comment on or make changes to this bug.