Closed Bug 1318635 Opened 3 years ago Closed 3 years ago

timeStamp of "dblclick" is always 0 in release build

Categories

(Core :: DOM: UI Events & Focus Handling, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: masayuki, Assigned: masayuki)

Details

Attachments

(1 file)

No description provided.
Smaug, would you like to review it? (I wonder, who can review around event handling during your vacation.)
I think we really should just focus on fixing bug 1026804. As far as I know, Chrome is already shipping highres timestamps. (but better to test and verify).

But yes, looks like the patch ensures both old and high res timestamp.
Comment on attachment 8812142 [details]
Bug 1318635 Set "dblclick" event's time and timeStamp to mouseup event's time and timeStamp

https://reviewboard.mozilla.org/r/94010/#review94156
Attachment #8812142 - Flags: review+
Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/dcf886cf73de
Set "dblclick" event's time and timeStamp to mouseup event's time and timeStamp r=smaug
https://hg.mozilla.org/mozilla-central/rev/dcf886cf73de
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Component: Event Handling → User events and focus handling
You need to log in before you can comment on or make changes to this bug.