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

RESOLVED FIXED in mozilla37

Status

()

defect
RESOLVED FIXED
5 years ago
4 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 8544909 [details] [diff] [review]
Mark virtual overridden functions as MOZ_OVERRIDE in dom/xul

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

::: dom/xul/templates/nsXULContentBuilder.cpp
@@ +90,5 @@
>      NS_NewXULContentBuilder(nsISupports* aOuter, REFNSIID aIID, void** aResult);
>  
>      nsXULContentBuilder();
>  
> +    void Traverse(nsCycleCollectionTraversalCallback &cb) const MOZ_OVERRIDE

..Callback& aCb

::: dom/xul/templates/nsXULTreeBuilder.cpp
@@ +188,5 @@
>      virtual nsresult
>      ReplaceMatch(nsIXULTemplateResult* aOldResult,
>                   nsTemplateMatch* aNewMatch,
>                   nsTemplateRule* aNewMatchRule,
> +                 void *aContext) MOZ_OVERRIDE;

void* aContext
Attachment #8544909 - Flags: review?(amarchesini) → review+
https://hg.mozilla.org/mozilla-central/rev/2dc5720003ee
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.