NS_OVERRIDE still used in xpcom/analysis/override.js

RESOLVED FIXED in mozilla21

Status

()

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: marco, Unassigned)

Tracking

17 Branch
mozilla21
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug])

Attachments

(1 attachment)

(Reporter)

Comment 1

6 years ago
See https://bugzilla.mozilla.org/show_bug.cgi?id=702877#c8, probably the entire script should be removed.
Whiteboard: [good first bug]

Comment 2

6 years ago
Something like this?
Attachment #701758 - Flags: review?(ehsan)

Comment 3

6 years ago
Comment on attachment 701758 [details] [diff] [review]
Bug 817387 - Remove last occurences of NS_OVERRIDE

Exactly!  Thanks for your patch.  :-)
Attachment #701758 - Flags: review?(ehsan) → review+
https://hg.mozilla.org/mozilla-central/rev/ae1eb238652b
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
You need to log in before you can comment on or make changes to this bug.