nsIPresShell.h (layout) brings in nsCoord.h (gfx)

ASSIGNED
Assigned to

Status

()

Core
Layout
P3
normal
ASSIGNED
17 years ago
9 years ago

People

(Reporter: Alec Flett, Assigned: Alec Flett)

Tracking

Trunk
Future
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

17 years ago
anyone who #includes nsIPresShell.h also has to bring in nsCoord.h, which
creates a dependency on gfx. 
This means many components bring in gfx unnecessariliy.
Solutions: 
- figure out why we need nsCoord.h & other gfx headers, see if we can change
APIs so they are not required
- move nsCoord.h & friends to a common place
(Assignee)

Updated

17 years ago
Blocks: 100107
(Assignee)

Updated

17 years ago
Status: NEW → ASSIGNED
Summary: nsIPresShell.h brings in gfx → nsIPresShell.h (layout) brings in nsCoord.h (gfx)
(Assignee)

Updated

17 years ago
Target Milestone: --- → mozilla0.9.5
(Assignee)

Comment 1

17 years ago
ok, it looks like nsCoord.h just defines:
typedef PRInt32 nscoord;

which is pretty silly. We want to keep the notion of nscoord as an abstract
type, but this should probably be moved out of gfx.
(Assignee)

Updated

17 years ago
Priority: -- → P3
(Assignee)

Updated

17 years ago
Target Milestone: mozilla0.9.5 → mozilla0.9.6
(Assignee)

Comment 2

17 years ago
not critical right now, moving out
Target Milestone: mozilla0.9.6 → mozilla0.9.7
(Assignee)

Updated

17 years ago
Depends on: 106686
(Assignee)

Updated

17 years ago
No longer blocks: 100107
(Assignee)

Updated

17 years ago
Target Milestone: mozilla0.9.7 → mozilla0.9.9
(Assignee)

Updated

17 years ago
Target Milestone: mozilla0.9.9 → mozilla1.1
(Assignee)

Updated

16 years ago
Target Milestone: mozilla1.1alpha → mozilla1.2alpha
(Assignee)

Updated

16 years ago
Target Milestone: mozilla1.2alpha → Future
QA Contact: chrispetersen → layout
You need to log in before you can comment on or make changes to this bug.