Use nsString.h less

RESOLVED FIXED in mozilla27

Status

()

Core
XPCOM
--
minor
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: neil@parkwaycc.co.uk, Assigned: neil@parkwaycc.co.uk)

Tracking

({regression})

Trunk
mozilla27
regression
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
From bug 918923: We should use it instead of nsStringGlue.h in places which only use the internal APIs.

However a number of public headers which would normally be available to users of the external API were included in the list, to wit:
imgRequest.h
nsICharsetConverterManger.idl
nsIDateTimeFormat.h
nsIPlatformCharset.h
nsIScriptContext.h
nsIWidget.h
(Assignee)

Comment 1

5 years ago
Created attachment 809445 [details] [diff] [review]
Proposed patch
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #809445 - Flags: review?(ehsan)

Comment 2

5 years ago
Comment on attachment 809445 [details] [diff] [review]
Proposed patch

Sorry if this broke your builds. :/
Attachment #809445 - Flags: review?(ehsan) → review+
https://hg.mozilla.org/mozilla-central/rev/e044880511db
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27

Comment 5

5 years ago
nsIParserService.h is another file that bug 918923 changed, and is now causing me grief in external builds :(
You need to log in before you can comment on or make changes to this bug.