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)
Core
DOM: UI Events & Focus Handling
Tracking
()
NEW
People
(Reporter: dbaron, Unassigned)
References
(Depends on 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
|
12.20 KB,
patch
|
bryner
:
review+
bryner
:
superreview+
|
Details | Diff | Splinter Review |
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.
| Reporter | ||
Comment 1•19 years ago
|
||
Attachment #192529 -
Flags: superreview?(bryner)
Attachment #192529 -
Flags: review?(bryner)
| Reporter | ||
Updated•19 years ago
|
Attachment #192529 -
Flags: superreview?(bryner)
Attachment #192529 -
Flags: superreview-
Attachment #192529 -
Flags: review?(bryner)
Attachment #192529 -
Flags: review-
| Reporter | ||
Comment 2•19 years ago
|
||
Attachment #192529 -
Attachment is obsolete: true
Attachment #192530 -
Flags: superreview?(bryner)
Attachment #192530 -
Flags: review?(bryner)
| Reporter | ||
Updated•19 years ago
|
Severity: normal → minor
Status: NEW → ASSIGNED
Priority: -- → P3
Whiteboard: [patch]
Target Milestone: --- → mozilla1.9alpha
Updated•19 years ago
|
Attachment #192530 -
Flags: superreview?(bryner)
Attachment #192530 -
Flags: superreview+
Attachment #192530 -
Flags: review?(bryner)
Attachment #192530 -
Flags: review+
| Reporter | ||
Comment 3•19 years ago
|
||
Fix checked in to trunk 2005-08-26 14:38 -0700.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
| Reporter | ||
Comment 5•19 years ago
|
||
I backed out the nsDOMEvent::SetEventType change until I can figure out why it causes bug 306210.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
| Reporter | ||
Comment 6•19 years ago
|
||
I think the event listener manager doesn't know how to store listeners unless I do the whole ArrayType / NS_EVENT_BITS dance.
| Reporter | ||
Comment 7•19 years ago
|
||
Most of the rest backed out (except the nsEventListenerManager change and some of the comment changes elsewhere).
| Reporter | ||
Updated•19 years ago
|
Assignee: dbaron → events
Status: REOPENED → NEW
Whiteboard: [patch]
| Reporter | ||
Updated•19 years ago
|
Target Milestone: mozilla1.9alpha → ---
Updated•15 years ago
|
Assignee: events → nobody
QA Contact: ian → events
| Assignee | ||
Updated•6 years ago
|
Component: Event Handling → User events and focus handling
Updated•2 years ago
|
Severity: minor → S4
You need to log in
before you can comment on or make changes to this bug.
Description
•