Don't use createEvent("TouchEvent") to check if TouchEvents are supported

RESOLVED FIXED in mozilla32

Status

()

defect
RESOLVED FIXED
5 years ago
3 months ago

People

(Reporter: smaug, Assigned: fabrice)

Tracking

({perf})

Trunk
mozilla32
x86_64
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Reporter

Description

5 years ago
http://mxr.mozilla.org/mozilla-central/source/dom/browser-element/BrowserElementPanning.js?rev=31631e0d4ada&mark=55-55#52
shows up a bit in the profiles when checking where time is spent during
<iframe mozbrowser> initialization.
I think '"TouchEvent" in content' would work just fine.
Assignee

Comment 1

5 years ago
I'm curious to know how much that changes in profiles!
Assignee: nobody → fabrice
Attachment #8415402 - Flags: review?(bugs)
Keywords: perf
Reporter

Comment 2

5 years ago
Comment on attachment 8415402 [details] [diff] [review]
touch-event.patch

This was a minor thing in profiles. But I happened to notice it in several
different cases.
Attachment #8415402 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/d81fe3476f86
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.