Rename EventTarget to EventListener

RESOLVED FIXED in 2.1 S9 (21Nov)

Status

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: allstars.chh, Unassigned)

Tracking

unspecified
2.1 S9 (21Nov)
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [p=1])

Attachments

(2 attachments)

I used the term EventTarget in Bug 1069177, however I think the correct one should be EventListener.
Created attachment 8521114 [details] [diff] [review]
Part 1: rename eventTarget to eventListener
Created attachment 8521115 [details] [diff] [review]
Part 2: rename nsINfcEventListener
(Assignee)

Updated

4 years ago
Attachment #8521114 - Flags: review?(dlee)
(Assignee)

Updated

4 years ago
Attachment #8521115 - Flags: review?(dlee)
Comment on attachment 8521114 [details] [diff] [review]
Part 1: rename eventTarget to eventListener

Review of attachment 8521114 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/nfc/nsINfcContentHelper.idl
@@ +104,2 @@
>     *
> +   * @param listener  An instance of the nsINfcDOMEventListener.

It would be better follow the same coding style for the comment in this IDL
Attachment #8521114 - Flags: review?(dlee) → review+

Updated

4 years ago
Attachment #8521115 - Flags: review?(dlee) → review+
You need to log in before you can comment on or make changes to this bug.