Closed Bug 1436743 Opened 2 years ago Closed 2 years ago

Use the docgroup to dispatch whenever possible

Categories

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

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla60
Tracking Status
firefox60 --- fixed

People

(Reporter: jdm, Assigned: jdm)

Details

Attachments

(1 file, 1 obsolete file)

In conjunction with bug 1436742, this yields more precise data when analyzing event distributions. I have a patch that I'll be attaching that deals with the straightforward cases.
Assignee: nobody → josh
Status: NEW → ASSIGNED
These are cases that were easy to convert when I searched for code that was explicitly dispatching to the current thread or to the tabgroup.
Comment on attachment 8949511 [details] [diff] [review]
Dispatch events via the docgroup rather than the tabgroup when possible

Review of attachment 8949511 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM :-)
Attachment #8949511 - Flags: review?(nika) → review+
Minor changes to ensure it actually builds.
Attachment #8949511 - Attachment is obsolete: true
Keywords: checkin-needed
Pushed by ncsoregi@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d9da3f3bf432
Dispatch events via the docgroup rather than the tabgroup when possible. r=mystor
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/d9da3f3bf432
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.