Closed Bug 977559 Opened 7 years ago Closed 7 years ago

We can initialize pointerId from system value


(Firefox for Metro Graveyard :: General, defect)

Not set


(Not tracked)

Firefox 31


(Reporter: alessarik, Assigned: alessarik)



(Whiteboard: [triage])


(1 file, 1 obsolete file)

User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:26.0) Gecko/20100101 Firefox/26.0 (Beta/Release)
Build ID: 20131205075310

Steps to reproduce:

Trunc version of FireFox dispatch mouse events with null pointerId

Expected results:

We should initialize pointer id as value from system
Attached patch not_null_pointer.diff (obsolete) — Splinter Review
When we create mouse events we can initialize pointerId as value from system
Blocks: metrobacklog
Whiteboard: [triage]
But does it really matter what is pointer id for Pointer generated from Mouse source?
if we make it somewhat different for One platform and all other platforms would continue having it == 0, it is not good.

Also requirement for this to be == 1 for mouse has been removed here:
Comment on attachment 8382993 [details] [diff] [review]

But here could be special case for "pen" device, which is usually generated into PressShell as MouseEvent...
I wonder is it ever possible to use Multiple Pen's at the same time?
Comment on attachment 8382993 [details] [diff] [review]

Form another point of view we may need this in order to initialize PointerId for Pen device...
But problem is that on Win32 UI we have Pen Source generated from Mouse event and there is no any info about PointerID... 
Also I haven't seen device which does support > 1 pen device at the same time.
So no reason to have HW generated unique ID there.
+ changes according with last version of source code

Despite of removing requirement for pointerId == 1 in case of mouse,
we can provide correct value, which we gets from system.
Attachment #8382993 - Attachment is obsolete: true
Attachment #8406015 - Flags: review?(oleg.romashin)
Attachment #8406015 - Flags: review?(nicklebedev37)
Attachment #8406015 - Flags: review?(mbrubeck)
Attachment #8406015 - Flags: review?(bugs)
Attachment #8406015 - Flags: review?(mbrubeck) → review+
Attachment #8406015 - Flags: review?(oleg.romashin) → review+
Attachment #8406015 - Flags: review?(bugs) → review+
Attachment #8406015 - Flags: review?(nicklebedev37)
Ever confirmed: true
Keywords: checkin-needed
Assignee: nobody → alessarik
Keywords: checkin-needed
Whiteboard: [triage] → [triage][fixed-in-fx-team]
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [triage][fixed-in-fx-team] → [triage]
Target Milestone: --- → Firefox 31
You need to log in before you can comment on or make changes to this bug.