From e76ff3bf160dc90968a5530b7477daaff67480b8 Mon Sep 17 00:00:00 2001 From: LukePulverenti Luke Pulverenti luke pulverenti Date: Tue, 18 Sep 2012 17:39:42 -0400 Subject: Improved image processing --- MediaBrowser.Controller/Drawing/ImageProcessor.cs | 27 +++++++++++++++-------- 1 file changed, 18 insertions(+), 9 deletions(-) (limited to 'MediaBrowser.Controller/Drawing/ImageProcessor.cs') diff --git a/MediaBrowser.Controller/Drawing/ImageProcessor.cs b/MediaBrowser.Controller/Drawing/ImageProcessor.cs index b7815750b..f9b6366c6 100644 --- a/MediaBrowser.Controller/Drawing/ImageProcessor.cs +++ b/MediaBrowser.Controller/Drawing/ImageProcessor.cs @@ -43,6 +43,8 @@ namespace MediaBrowser.Controller.Drawing thumbnail = new Bitmap(newSize.Width, newSize.Height, originalImage.PixelFormat); } + thumbnail.MakeTransparent(); + // Preserve the original resolution thumbnail.SetResolution(originalImage.HorizontalResolution, originalImage.VerticalResolution); @@ -56,14 +58,21 @@ namespace MediaBrowser.Controller.Drawing thumbnailGraph.DrawImage(originalImage, 0, 0, newSize.Width, newSize.Height); + ImageFormat outputFormat = originalImage.RawFormat; + // Run Kernel image processors if (Kernel.Instance.ImageProcessors.Any()) { - ExecuteAdditionalImageProcessors(thumbnail, thumbnailGraph, entity, imageType, imageIndex); + ExecuteAdditionalImageProcessors(originalImage, thumbnail, thumbnailGraph, entity, imageType, imageIndex); + + if (Kernel.Instance.ImageProcessors.Any(i => i.RequiresTransparency)) + { + outputFormat = ImageFormat.Png; + } } // Write to the output stream - SaveImage(originalImage.RawFormat, thumbnail, toStream, quality); + SaveImage(outputFormat, thumbnail, toStream, quality); thumbnailGraph.Dispose(); thumbnail.Dispose(); @@ -78,7 +87,7 @@ namespace MediaBrowser.Controller.Drawing /// The entity that owns the image /// The image type /// The image index (currently only used with backdrops) - private static void ExecuteAdditionalImageProcessors(Bitmap bitmap, Graphics graphics, BaseEntity entity, ImageType imageType, int imageIndex) + private static void ExecuteAdditionalImageProcessors(Image originalImage, Bitmap bitmap, Graphics graphics, BaseEntity entity, ImageType imageType, int imageIndex) { var baseItem = entity as BaseItem; @@ -86,33 +95,33 @@ namespace MediaBrowser.Controller.Drawing { foreach (var processor in Kernel.Instance.ImageProcessors) { - processor.ProcessImage(bitmap, graphics, baseItem, imageType, imageIndex); + processor.ProcessImage(originalImage, bitmap, graphics, baseItem, imageType, imageIndex); } } else { foreach (var processor in Kernel.Instance.ImageProcessors) { - processor.ProcessImage(bitmap, graphics, entity); + processor.ProcessImage(originalImage, bitmap, graphics, entity); } } } - public static void SaveImage(ImageFormat originalImageRawFormat, Image newImage, Stream toStream, int? quality) + public static void SaveImage(ImageFormat outputFormat, Image newImage, Stream toStream, int? quality) { // Use special save methods for jpeg and png that will result in a much higher quality image // All other formats use the generic Image.Save - if (ImageFormat.Jpeg.Equals(originalImageRawFormat)) + if (ImageFormat.Jpeg.Equals(outputFormat)) { SaveJpeg(newImage, toStream, quality); } - else if (ImageFormat.Png.Equals(originalImageRawFormat)) + else if (ImageFormat.Png.Equals(outputFormat)) { newImage.Save(toStream, ImageFormat.Png); } else { - newImage.Save(toStream, originalImageRawFormat); + newImage.Save(toStream, outputFormat); } } -- cgit v1.2.3