Closed Bug 1191305 Opened 7 years ago Closed 7 years ago

Alphabetize some includes

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla42
Tracking Status
firefox42 --- fixed

People

(Reporter: ayg, Assigned: ayg)

References

Details

Attachments

(1 file)

(In reply to Nathan Froyd [:froydnj][:nfroyd] from bug 1190823 comment #4)
> It would be great if you could fix the alphabetization of the headers in
> some of the files you modified, but since we don't have decent tools to do
> that, a follow-up bug for those is fine.
Attached patch PatchSplinter Review
Green try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=3b4fbb296381

I split the includes into sections by directory.
Attachment #8643763 - Flags: review?(nfroyd)
Comment on attachment 8643763 [details] [diff] [review]
Patch

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

r=me assuming nothing complains about the reordering on try.
Attachment #8643763 - Flags: review?(nfroyd) → review+
https://hg.mozilla.org/mozilla-central/rev/975e8b161839
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.