Closed
Bug 198333
Opened 22 years ago
Closed 22 years ago
Can we get rid of nsDispatchListener?
Categories
(SeaMonkey :: UI Design, defect)
SeaMonkey
UI Design
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jag+mozilla, Assigned: jag+mozilla)
Details
Attachments
(1 file)
|
14.76 KB,
patch
|
darin.moz
:
review+
bryner
:
superreview+
asa
:
approval1.4+
|
Details | Diff | Splinter Review |
The only method on that class is |virtual void AfterDispatch();|, and the one
implementation of it (in webshell/tests/viewer/nsViewerApp) is empty.
|
Assignee | |
Comment 1•22 years ago
|
||
|
|
Assignee | |
Comment 2•22 years ago
|
||
|
||
Comment 3•22 years ago
|
||
i know nothing of this interface myself, but it certainly does look like a good
candidate for removal. i'll bite... r/sr=darin
|
|
||
Updated•22 years ago
|
Attachment #117808 -
Flags: superreview+
|
|
||
Comment 4•22 years ago
|
||
Comment on attachment 117808 [details] [diff] [review]
Remove nsDispatchListener
r=darin
Attachment #117808 -
Flags: review+
|
|
Assignee | |
Updated•22 years ago
|
Attachment #117808 -
Flags: approval1.4?
|
||
Comment 5•22 years ago
|
||
Comment on attachment 117808 [details] [diff] [review]
Remove nsDispatchListener
a=asa (on behalf of drivers) for checkin to 1.4.
Attachment #117808 -
Flags: approval1.4? → approval1.4+
|
|
Assignee | |
Comment 6•22 years ago
|
||
Checked in (note that I had to fix the widget/photon/src/nsAppShell.h too).
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
|
||
Updated•21 years ago
|
Product: Core → Mozilla Application Suite
You need to log in
before you can comment on or make changes to this bug.
Description
•