The default bug view has changed. See this FAQ.

Kill txAtoms.h and s/txXPathAtoms|txXMLAtoms|txXSLTAtoms|txHTMLAtoms/nsGkAtoms/g

RESOLVED FIXED in mozilla9

Status

()

Core
XSLT
--
minor
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: emorley, Assigned: emorley)

Tracking

Trunk
mozilla9
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

In bug 683416, support for standalone xslt was removed, meaning that some of the typedefs that used to be in #ifndef TX_EXE blocks are now redundant.

Sicking in bug 683416 comment 9:
> It'd be awesome to also kill txAtoms.h and do 
> s/txXPathAtoms|txXMLAtoms|txXSLTAtoms|txHTMLAtoms/nsGkAtoms/g
Flags: in-testsuite-
Created attachment 560777 [details] [diff] [review]
Patch v1

- Hg removes txAtoms.h
- s/#include "txAtoms.h"/#include "nsGkAtoms.h"/g
- s/txXPathAtoms|txXMLAtoms|txXSLTAtoms|txHTMLAtoms/nsGkAtoms/g

Builds fine locally, sent to try:
https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=6d1bcb9a507a
Attachment #560777 - Flags: review?(jonas)
Attachment #560777 - Flags: review?(jonas) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/1d5cdab249a9
Target Milestone: --- → mozilla9
https://hg.mozilla.org/mozilla-central/rev/1d5cdab249a9
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.