Don't include nsString.h in nsIXPathEvaluatorInternal.h

RESOLVED FIXED in mozilla13

Status

()

Core
XPCOM
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: imphil, Assigned: imphil)

Tracking

Trunk
mozilla13
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

6 years ago
Created attachment 595024 [details] [diff] [review]
patch

XForms includes this file and it's using the external string API. This was introduced with bug 693949.
(Assignee)

Updated

6 years ago
Attachment #595024 - Flags: review?(benjamin)
Comment on attachment 595024 [details] [diff] [review]
patch

This seems silly, but ok.
Attachment #595024 - Flags: review?(benjamin) → review+
(Assignee)

Comment 2

5 years ago
Created attachment 595434 [details] [diff] [review]
patch for checkin; r=bsmedberg

could someone check this in for me, please?
Assignee: nobody → mail
Status: NEW → ASSIGNED
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
(Assignee)

Comment 3

5 years ago
Created attachment 595436 [details] [diff] [review]
patch for checkin; r=bsmedberg

uh, the other change should have not been part of this patch.
Attachment #595434 - Attachment is obsolete: true
http://hg.mozilla.org/integration/mozilla-inbound/rev/abcf47b9300d
Keywords: checkin-needed
Target Milestone: --- → mozilla13

Comment 5

5 years ago
https://hg.mozilla.org/mozilla-central/rev/abcf47b9300d
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.