nsSpaceManager doesn't need an XPCOM interface

RESOLVED FIXED in mozilla0.9.6

Status

()

defect
RESOLVED FIXED
18 years ago
17 years ago

People

(Reporter: dbaron, Assigned: dbaron)

Tracking

(Blocks 1 bug)

Trunk
mozilla0.9.6
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

nsSpaceManager is entirely internal to layout -- it doesn't need an XPCOM
interface (nsISpaceManager).  If the interface were removed, we could even use
some inline functions for some of the simple things this does (particularly the
ones I'm going to add for bug 86947).

Updated

18 years ago
Target Milestone: --- → mozilla0.9.6

Comment 3

18 years ago
Comment on attachment 55106 [details] [diff] [review]
cleaned-up patch

sr=waterson
Attachment #55106 - Flags: superreview+
+  // Get the bouding rect of the trapezoid

typo alert.

Also, what does "semantic compare" mean?

/be
I have to admit that I haven't even read all the code that I moved from one file
to the other. :-)

Comment 6

18 years ago
Comment on attachment 55106 [details] [diff] [review]
cleaned-up patch

r=jag
Attachment #55106 - Flags: review+
Checked in 2001-10-25 22:05/06 PDT.
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.