Open Bug 91703 Opened 24 years ago Updated 2 years ago

vague comment about consume/ignore return values for listeners

Categories

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

x86
Windows NT
defect

Tracking

()

People

(Reporter: jruderman, Unassigned)

References

()

Details

Several comments say "returns whether the event was consumed or ignored".  
These comments should be more specific: what should implementers return to 
consume the event?  What should they return to ignore it?

(Also, a nearby comment mentions the non-existent nsIDOMEvent.h.)

Several of the files with the confusing comment:
- dom/public/coreEvents/nsIDOMFocusListener.h
- dom/public/coreEvents/nsIDOMMouseListener.h
- dom/public/coreEvents/nsIDOMKeyListener.h

Here's what the comment looks like:

/**
 * Processes a mouse down event
 * @param aMouseEvent @see nsIDOMEvent.h
 * @returns whether the event was consumed or ignored. @see nsresult
 */
NS_IMETHOD MouseDown(nsIDOMEvent* aMouseEvent) = 0;
Target Milestone: --- → mozilla0.9.8
Target Milestone: mozilla0.9.8 → mozilla0.9.9
Target Milestone: mozilla0.9.9 → mozilla1.0
Target Milestone: mozilla1.0 → mozilla1.1
QA Contact: madhur → rakeshmishra
QA Contact: rakeshmishra → trix
.
Assignee: joki → saari
QA Contact: trix → ian
retargeting
Target Milestone: mozilla1.1alpha → Future
Assignee: saari → events
Target Milestone: Future → ---
Assignee: events → nobody
QA Contact: ian → events
Component: Event Handling → User events and focus handling
Severity: trivial → S4
You need to log in before you can comment on or make changes to this bug.