Closed Bug 1038536 Opened 6 years ago Closed 6 years ago

Flatten away public/src subdirectories under image/

Categories

(Core :: ImageLib, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla41
Tracking Status
firefox41 --- fixed

People

(Reporter: poiru, Assigned: poiru)

Details

Attachments

(4 files)

This is similar to e.g. bug 1028559 and bug 946065.

Seth, would you be OK with this?
Flags: needinfo?(seth)
Summary: Flatten away public/src subdirectories under caps/ → Flatten away public/src subdirectories under image/
Attachment #8461590 - Flags: review?(seth)
Flags: needinfo?(seth)
Attachment #8461591 - Flags: review?(seth)
I'm deliberately holding off on r+'ing this until some existing bugs land, to avoid any rebasing issues.
(In reply to Seth Fowler [:seth] from comment #3)
> I'm deliberately holding off on r+'ing this until some existing bugs land,
> to avoid any rebasing issues.

Have these bugs landed yet?
Status: NEW → ASSIGNED
Flags: needinfo?(seth)
No, unfortunately. imagelib is in a really heavy state of churn right now. But maybe that's actually an argument that we should go ahead and just land this; things probably won't get better for a couple of months at least.
Flags: needinfo?(seth)
Attachment #8461590 - Flags: review?(seth) → review+
Attachment #8461591 - Flags: review?(seth) → review+
I went ahead and landed the image/public/ bit. Let me know if/when I can land the image/src/ patch.
Keywords: leave-open
The header guard in ImageLogging.h is out of compliance with the Mozilla coding style due to its relocation.  It should now be

mozilla_image_ImageLogging_h
instead of
mozilla_image_public_ImageLogging_h

I'll post a separate patch for image/src headers, to be applied after the image/src files are relocated.
Assignee: birunthan → glennrp+bmo
Attachment #8603117 - Flags: review?(seth)
Assignee: glennrp+bmo → birunthan
Comment on attachment 8603117 [details] [diff] [review]
v00-1038356-update-build-headers-after-flatten

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

Thanks, Glenn!
Attachment #8603117 - Flags: review?(seth) → review+
Attachment #8603117 - Flags: checkin?
(In reply to Birunthan Mohanathas [:poiru] from comment #7)
> I went ahead and landed the image/public/ bit. Let me know if/when I can
> land the image/src/ patch.

Now that we're past the Gecko 40 merge to Aurora, I think it's a good time. Go for it, whenever you're ready.
Comment on attachment 8605553 [details] [diff] [review]
Flatten image/decoders/icon/qt/public/ directory

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

Looks good!
Attachment #8605553 - Flags: review?(seth) → review+
Keywords: leave-open
Attachment #8603117 - Flags: checkin?
Flags: needinfo?(birunthan)
https://hg.mozilla.org/mozilla-central/rev/f0b9343094e0
https://hg.mozilla.org/mozilla-central/rev/7701abd92485
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in before you can comment on or make changes to this bug.