Closed
Bug 1066237
Opened 7 years ago
Closed 7 years ago
MaxTouchPoints variable may return invalid value on the win7 platform
Categories
(Core :: DOM: Events, defect)
Tracking
()
RESOLVED
FIXED
mozilla35
People
(Reporter: nl, Assigned: nl)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
989 bytes,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
Please read the following thread for more info: http://lists.w3.org/Archives/Public/public-pointer-events/2014JulSep/0116.html
Assignee | ||
Updated•7 years ago
|
Comment 1•7 years ago
|
||
http://mxr.mozilla.org/mozilla-central/source/dom/webidl/Navigator.webidl#146 It doesn't
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
Comment 2•7 years ago
|
||
Though, I guess we want to change the implementation anyhow.
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Assignee | ||
Comment 3•7 years ago
|
||
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.
Assignee | ||
Comment 4•7 years ago
|
||
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)
Updated•7 years ago
|
Attachment #8503222 -
Flags: review?(bugs) → review+
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Comment 5•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/7d6eb07eb884
Keywords: checkin-needed
Comment 6•7 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/7d6eb07eb884
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
You need to log in
before you can comment on or make changes to this bug.
Description
•