Merge pull request #7569 from MarchingCube/win32-filepicker-no-exceptions

Avoid using COM exceptions for dialog control flow.

src/Windows/Avalonia.Win32/SystemDialogImpl.cs

@@ -89,7 +89,16 @@ namespace Avalonia.Win32
                         }
                     }
 
-                    frm.Show(hWnd);
+                    var showResult = frm.Show(hWnd);
+
+                    if ((uint)showResult == (uint)UnmanagedMethods.HRESULT.E_CANCELLED)
+                    {
+                        return result;
+                    } 
+                    else if ((uint)showResult != (uint)UnmanagedMethods.HRESULT.S_OK)
+                    {
+                        throw new Win32Exception(showResult);
+                    }
 
                     if (openDialog?.AllowMultiple == true)
                     {
@@ -118,10 +127,6 @@ namespace Avalonia.Win32
                 }
                 catch (COMException ex)
                 {
-                    if ((uint)ex.HResult == (uint)UnmanagedMethods.HRESULT.E_CANCELLED)
-                    {
-                        return result;
-                    }
                     throw new Win32Exception(ex.HResult);
                 }
             })!;
@@ -161,7 +166,17 @@ namespace Avalonia.Win32
                         }
                     }
 
-                    frm.Show(hWnd);
+                    var showResult = frm.Show(hWnd);
+
+                    if ((uint)showResult == (uint)UnmanagedMethods.HRESULT.E_CANCELLED)
+                    {
+                        return result;
+                    }
+                    else if ((uint)showResult != (uint)UnmanagedMethods.HRESULT.S_OK)
+                    {
+                        throw new Win32Exception(showResult);
+                    }
+
                     if (frm.Result is not null)
                     {
                         result = GetAbsoluteFilePath(frm.Result);
@@ -171,10 +186,6 @@ namespace Avalonia.Win32
                 }
                 catch (COMException ex)
                 {
-                    if ((uint)ex.HResult == (uint)UnmanagedMethods.HRESULT.E_CANCELLED)
-                    {
-                        return result;
-                    }
                     throw new Win32Exception(ex.HResult);
                 }
             });

src/Windows/Avalonia.Win32/Win32Com/win32.idl

@@ -112,7 +112,7 @@ interface IShellItemArray : IUnknown
 interface IModalWindow : IUnknown
 {
     [local]
-    HRESULT Show(
+    int Show(
         [in, unique] HWND hwndOwner);
 }