nsHTMLEditor.cpp contains unnecessary includes

RESOLVED FIXED in mozilla5

Status

()

Core
Editor
RESOLVED FIXED
9 years ago
7 years ago

People

(Reporter: arno renevier, Assigned: arno renevier)

Tracking

unspecified
mozilla5
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

3.18 KB, patch
peterv
: review+
peterv
: superreview+
Benjamin Smedberg
: approval2.0-
Details | Diff | Splinter Review
(Assignee)

Description

9 years ago
Hi, 
nsHTMLEditor.cpp contains unnessary includes. I could remove 27 of them and have the code still compile. Patch coming soon.
(Assignee)

Comment 1

9 years ago
Created attachment 382315 [details] [diff] [review]
patch v1

I discovered that accidentaly, but may some more files need cleanup.
Assignee: nobody → arno
(Assignee)

Updated

9 years ago
Attachment #382315 - Flags: review?(peterv)
Attachment #382315 - Flags: superreview+
Attachment #382315 - Flags: review?(peterv)
Attachment #382315 - Flags: review+
So this is good for checkin right?

Comment 3

8 years ago
Comment on attachment 382315 [details] [diff] [review]
patch v1

(In reply to comment #2)
> So this is good for checkin right?

Once it gets approved.
Attachment #382315 - Flags: approval2.0?

Comment 4

7 years ago
To keep the tree quiet, let's wait until after FF4 branches.

Updated

7 years ago
Attachment #382315 - Flags: approval2.0? → approval2.0-

Updated

7 years ago
Whiteboard: [ready-for-post-2.0]

Comment 5

7 years ago
http://hg.mozilla.org/projects/cedar/rev/8603881f7308
Whiteboard: [ready-for-post-2.0] → fixed-in-cedar

Comment 6

7 years ago
http://hg.mozilla.org/mozilla-central/rev/8603881f7308

Thanks a lot for your patch, Arno!
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Whiteboard: fixed-in-cedar
Target Milestone: --- → mozilla2.2
You need to log in before you can comment on or make changes to this bug.