Closed Bug 1003836 Opened 6 years ago Closed 6 years ago

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

Categories

(Core :: DOM: Core & HTML, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla32

People

(Reporter: smaug, Assigned: fabrice)

References

Details

(Keywords: perf)

Attachments

(1 file)

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.
I'm curious to know how much that changes in profiles!
Assignee: nobody → fabrice
Attachment #8415402 - Flags: review?(bugs)
Keywords: perf
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: 6 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.