Open Bug 304462 Opened 19 years ago Updated 2 years ago

popup blocked events don't have a message constant

Categories

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

defect

Tracking

()

People

(Reporter: dbaron, Unassigned)

References

(Depends on 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

In bug 282103 comment 56, bryner says we fixed all event classes to have message
constants.  This isn't actually true for popupblocked events.  I had a fix for
this in some versions of my patch to bug 291725, but I didn't leave it in the
final version.
Attached patch patch (obsolete) — Splinter Review
Attachment #192529 - Flags: superreview?(bryner)
Attachment #192529 - Flags: review?(bryner)
Attachment #192529 - Flags: superreview?(bryner)
Attachment #192529 - Flags: superreview-
Attachment #192529 - Flags: review?(bryner)
Attachment #192529 - Flags: review-
Attached patch patchSplinter Review
Attachment #192529 - Attachment is obsolete: true
Attachment #192530 - Flags: superreview?(bryner)
Attachment #192530 - Flags: review?(bryner)
Severity: normal → minor
Status: NEW → ASSIGNED
Priority: -- → P3
Whiteboard: [patch]
Target Milestone: --- → mozilla1.9alpha
Attachment #192530 - Flags: superreview?(bryner)
Attachment #192530 - Flags: superreview+
Attachment #192530 - Flags: review?(bryner)
Attachment #192530 - Flags: review+
Fix checked in to trunk 2005-08-26 14:38 -0700.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Depends on: 306210
This caused bug 306210
I backed out the nsDOMEvent::SetEventType change until I can figure out why it
causes bug 306210.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
I think the event listener manager doesn't know how to store listeners unless I
do the whole ArrayType / NS_EVENT_BITS dance.
Most of the rest backed out (except the nsEventListenerManager change and some
of the comment changes elsewhere).
Assignee: dbaron → events
Status: REOPENED → NEW
Whiteboard: [patch]
Target Milestone: mozilla1.9alpha → ---
Assignee: events → nobody
QA Contact: ian → events
Component: Event Handling → User events and focus handling
Severity: minor → S4
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: