Mark virtual overridden functions as MOZ_OVERRIDE in dom/events; r=baku

RESOLVED FIXED in mozilla37

Status

()

defect
RESOLVED FIXED
5 years ago
5 months ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

unspecified
mozilla37
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

No description provided.
Assignee: nobody → ehsan
Comment on attachment 8544907 [details] [diff] [review]
Mark virtual overridden functions as MOZ_OVERRIDE in dom/events

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

::: dom/events/Event.h
@@ +292,5 @@
>    NS_FORWARD_NSIDOMEVENT(Event::)
>  
>  #define NS_FORWARD_NSIDOMEVENT_NO_SERIALIZATION_NO_DUPLICATION(_to) \
> +  NS_IMETHOD GetType(nsAString& aType) MOZ_OVERRIDE { return _to GetType(aType); } \
> +  NS_IMETHOD GetTarget(nsIDOMEventTarget * *aTarget) MOZ_OVERRIDE { return _to GetTarget(aTarget); } \

** can you fix the & and * in this list of methods?
Attachment #8544907 - Flags: review?(amarchesini) → review+
https://hg.mozilla.org/mozilla-central/rev/5d801822e67f
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.