Closed Bug 821993 Opened 12 years ago Closed 6 years ago

nsDOMDeviceStorage::RemoveEventListener doesn't pass through aCapture

Categories

(Core :: DOM: Device Interfaces, defect)

Product:
Core
Component:
DOM: Device Interfaces
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED INCOMPLETE

People

(Reporter: Ms2ger, Unassigned)

References

Details

http://mxr.mozilla.org/mozilla-central/source/dom/devicestorage/nsDeviceStorage.cpp#2270

2270 NS_IMETHODIMP
2271 nsDOMDeviceStorage::RemoveEventListener(const nsAString & aType,
2272                                         nsIDOMEventListener *aListener,
2273                                         bool aUseCapture)
2274 {
2275   nsDOMEventTargetHelper::RemoveEventListener(aType, aListener, false);

That 'false' should probably be 'aUseCapture'.
Doug, can you confirm?
Flags: needinfo?(doug.turner)
not really sure, but I suspect you're correct.
Flags: needinfo?(doug.turner)
Yes. And when fixing this, please add a test.
FxOS/Gonk has been removed from the codebase. Mass-invalidating FxOS related Device Interface bugs to clean up the component. 

If I incorrectly invalidated something, please let me know.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
Bulk correction of resolution of B2G bugs to INCOMPLETE.
Resolution: INVALID → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.