Closed Bug 685786 Opened 8 years ago Closed 8 years ago

give nsPresShell a .h file

Categories

(Core :: Layout, defect, P4)

defect

Tracking

()

RESOLVED FIXED
mozilla9

People

(Reporter: dbaron, Assigned: dbaron)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
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+
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
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.