give nsPresShell a .h file

RESOLVED FIXED in mozilla9

Status

()

Core
Layout
P4
normal
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: dbaron, Assigned: dbaron)

Tracking

Trunk
mozilla9
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

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

nsPresShell currently has the class definition in the .cpp file rather than in a .h file.  Having the class definition in a .h file is the first step towards merging nsPresShell into nsPresContext step-by-step.

I've had this patch in my tree for a while; may as well land it.
Attachment #559383 - Flags: review?(roc)
Comment on attachment 559383 [details] [diff] [review]
patch

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

You could reduce the diff and preserve some history using "hg copy", I guess.
Attachment #559383 - Flags: review?(roc) → review+
(Assignee)

Comment 2

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/ba84d2ccc09c
Component: Layout: Tables → Layout
Priority: -- → P4
QA Contact: layout.tables → layout
Target Milestone: --- → mozilla9
(In reply to David Baron [:dbaron] from comment #2)
> http://hg.mozilla.org/integration/mozilla-inbound/rev/ba84d2ccc09c

and now on m-c
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.