Juliata/filetypes (#22538)

* Using the same list of file extensions as Lightbox's AppxManifest, and ensuring we convert file extension to lowercase

* Add IsFileTypeSupported to IPreviewer

* respond to PR comments

src/modules/peek/Peek.Common/Models/File.cs

@@ -21,7 +21,7 @@ public File(string path)
 
         public string Path { get; init; }
 
-        public string Extension => System.IO.Path.GetExtension(Path);
+        public string Extension => System.IO.Path.GetExtension(Path).ToLower();
 
         public async Task<StorageFile> GetStorageFileAsync()
         {

src/modules/peek/Peek.FilePreviewer/Previewers/IPreviewer.cs

@@ -4,6 +4,7 @@
 
 namespace Peek.FilePreviewer.Previewers
 {
+    using System;
     using System.ComponentModel;
     using System.Threading.Tasks;
     using Windows.Foundation;
@@ -12,6 +13,8 @@ public interface IPreviewer : INotifyPropertyChanged
     {
         bool IsPreviewLoaded { get; }
 
+        public static bool IsFileTypeSupported(string fileExt) => throw new NotImplementedException();
+
         public Task<Size> GetPreviewSizeAsync();
 
         Task LoadPreviewAsync();

src/modules/peek/Peek.FilePreviewer/Previewers/ImagePreviewer/ImagePreviewer.cs

@@ -5,6 +5,7 @@
 namespace Peek.FilePreviewer.Previewers
 {
     using System;
+    using System.Collections.Generic;
     using System.Drawing.Imaging;
     using System.IO;
     using System.Threading;
@@ -174,5 +175,75 @@ private static async Task<BitmapSource> GetBitmapFromHBitmapAsync(IntPtr hbitmap
                 NativeMethods.DeleteObject(hbitmap);
             }
         }
+
+        public static bool IsFileTypeSupported(string fileExt)
+        {
+            return _supportedFileTypes.Contains(fileExt);
+        }
+
+        private static readonly HashSet<string> _supportedFileTypes = new HashSet<string>
+        {
+                // Image types
+                ".bmp",
+                ".gif",
+                ".jpg",
+                ".jfif",
+                ".jfi",
+                ".jif",
+                ".jpeg",
+                ".jpe",
+                ".png",
+                ".tif",
+                ".tiff",
+                ".dib",
+
+                // ".heic", // Error in System.Drawing.Image.FromHbitmap(hbitmap);
+                ".heif",
+                ".hif",
+                ".avif",
+                ".jxr",
+                ".wdp",
+                ".ico",
+                ".thumb",
+
+                // Raw types
+                ".arw",
+                ".cr2",
+
+                // ".crw", // Error in WICImageFactory.CreateDecoderFromFilename
+                // ".erf", // Error in WICImageFactory.CreateDecoderFromFilename
+                ".kdc",
+                ".mrw",
+                ".nef",
+                ".nrw",
+                ".orf",
+                ".pef",
+                ".raf",
+                ".raw",
+                ".rw2",
+                ".rwl",
+                ".sr2",
+                ".srw",
+                ".srf",
+                ".dcs",
+                ".dcr",
+                ".drf",
+                ".k25",
+                ".3fr",
+                ".ari",
+                ".bay",
+                ".cap",
+                ".iiq",
+                ".eip",
+                ".fff",
+                ".mef",
+                ".mdc",
+                ".mos",
+                ".R3D",
+                ".rwz",
+                ".x3f",
+                ".ori",
+                ".cr3",
+        };
     }
 }

src/modules/peek/Peek.FilePreviewer/Previewers/PreviewerFactory.cs

@@ -10,24 +10,13 @@ public class PreviewerFactory
     {
         public IPreviewer? Create(File file)
         {
-            // TODO: investigate performance of reflection to resolve previewer type
-            switch (file.Extension)
+            if (ImagePreviewer.IsFileTypeSupported(file.Extension))
             {
-                case ".bmp":
-                case ".gif":
-                case ".jpg":
-                case ".jfif":
-                case ".jfi":
-                case ".jif":
-                case ".jpeg":
-                case ".jpe":
-                case ".png":
-                case ".tif":
-                case ".tiff":
-                    return new ImagePreviewer(file);
-                default:
-                    return null;
+                return new ImagePreviewer(file);
             }
+
+            // Other previewer types check their supported file types here
+            return null;
         }
     }
 }