662946 - Remove a bunch of headers from INSTALLED_HEADERS, making changes necessary to do so

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jeff Walden [:Waldo]]

Attachment: Patch

Inspired by bug 662231, I went on a mini-rampage pruning INSTALLED_HEADERS by almost 20%.  Some of the entries in the list can be flat-out removed with no other changes.  Others need a little trimming in Mozilla code to remove them.  Others need (dead) callers to be killed in order to remove then-pointless inclusions.  Some need some forward-declares to avoid inclusions.  But anyway, all the removals here were pretty easy, and they required no deep surgery to disconnect XPConnect and jsobj.h or jsscope.h or jscntxt.h or anything similarly awful.  I'll leave that for other people and other times.

Try-building has not yet given this a stamp of approval:

http://tbpl.mozilla.org/?tree=Try&rev=32b4723ce26c

But a local build has, so I'm posting this in anticipation of being able to r? it shortly.

Comment 1

[Jim Blandy :jimb]

Hi, Jeff. I'll be able to review this and your other patches on Monday, but for today I have some Debugger stuff I'm really hoping to wrap up before Jason gets back from vacation.

Comment 2

[Jim Blandy :jimb]

Comment on attachment 538125 [details] [diff] [review]
Patch

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

r+, including parts I'm not a reviewer for, since it's dead code.

Comment 3

[Jeff Walden [:Waldo]]

http://hg.mozilla.org/tracemonkey/rev/ce077af8195c

Comment 4

[Franck]

Note that the bug 615621 is probably related to this one.

Comment 5

[Chris Leary [:cdleary] (not checking bugmail)]

cdleary-bot mozilla-central merge info:
http://hg.mozilla.org/mozilla-central/rev/ce077af8195c