Skip to content

GeneratedSampleOptionRenderer dependency property fails to bind under AoT, crashes sample galleryΒ #213

Closed

Description

@Arlodotexe

Describe the bug

With the latest updates to WinAppSDK 1.6 with AoT, we're seeing an exception coming from the WASDK build of our gallery for components using a generated options pane.

   at WinRT.ExceptionHelpers.<ThrowExceptionForHR>g__Throw|38_0(Int32 hr)
   at ABI.Microsoft.UI.Xaml.Controls.IItemsControlMethods.set_ItemsSource(IObjectReference _obj, Object value)
   at Microsoft.UI.Xaml.Controls.ItemsControl.set_ItemsSource(Object value)
   at CommunityToolkit.App.Shared.Renderers.GeneratedSampleOptionsRenderer.XamlBindingSetters.Set_Microsoft_UI_Xaml_Controls_ItemsControl_ItemsSource(ItemsControl obj, Object value, String targetNullValue) in /_/ProjectHeads/AllComponents/Wasdk/obj/x64/Debug/net8.0-windows10.0.22621.0/win-x64/Renderers/GeneratedSampleOptionsRenderer.g.cs:line 27
   at CommunityToolkit.App.Shared.Renderers.GeneratedSampleOptionsRenderer.GeneratedSampleOptionsRenderer_obj1_Bindings.Update_SampleOptions(IEnumerable`1 obj, Int32 phase) in /_/ProjectHeads/AllComponents/Wasdk/obj/x64/Debug/net8.0-windows10.0.22621.0/win-x64/Renderers/GeneratedSampleOptionsRenderer.g.cs:line 2011
   at CommunityToolkit.App.Shared.Renderers.GeneratedSampleOptionsRenderer.GeneratedSampleOptionsRenderer_obj1_Bindings.Update_(GeneratedSampleOptionsRenderer obj, Int32 phase) in /_/ProjectHeads/AllComponents/Wasdk/obj/x64/Debug/net8.0-windows10.0.22621.0/win-x64/Renderers/GeneratedSampleOptionsRenderer.g.cs:line 2000
   at CommunityToolkit.App.Shared.Renderers.GeneratedSampleOptionsRenderer.GeneratedSampleOptionsRenderer_obj1_Bindings.Update() in /_/ProjectHeads/AllComponents/Wasdk/obj/x64/Debug/net8.0-windows10.0.22621.0/win-x64/Renderers/GeneratedSampleOptionsRenderer.g.cs:line 1956
   at CommunityToolkit.App.Shared.Renderers.GeneratedSampleOptionsRenderer.GeneratedSampleOptionsRenderer_obj1_Bindings.Initialize() in /_/ProjectHeads/AllComponents/Wasdk/obj/x64/Debug/net8.0-windows10.0.22621.0/win-x64/Renderers/GeneratedSampleOptionsRenderer.g.cs:line 1950
   at CommunityToolkit.App.Shared.Renderers.GeneratedSampleOptionsRenderer.GeneratedSampleOptionsRenderer_obj1_Bindings.Loading(FrameworkElement src, Object data) in /_/ProjectHeads/AllComponents/Wasdk/obj/x64/Debug/net8.0-windows10.0.22621.0/win-x64/Renderers/GeneratedSampleOptionsRenderer.g.cs:line 1989
   at WinRT._EventSource_global__Windows_Foundation_TypedEventHandler_global__Microsoft_UI_Xaml_FrameworkElement__object_.EventState.<GetEventInvoke>b__1_0(FrameworkElement sender, Object args)
   at WinRT.GenericTypeInstantiations.Windows_Foundation_TypedEventHandler_2_Microsoft_UI_Xaml_FrameworkElement__object.Do_Abi_Invoke(IntPtr thisPtr, IntPtr sender, IntPtr args)

Steps to reproduce

1. Clone the toolkit repo
2. Generate and open the wasdk solution: `./tooling/GenerateAllSolution.ps1 -winui 3 -heads wasdk -multitargets wasdk`
3. Build and deploy the gallery CommunityToolkit.App.Wasdk
4. Navigate to a component that uses generated sample options, such as SettingsControl.

Expected behavior

No crash.

Screenshots

No response

Code Platform

  • UWP
  • WinAppSDK / WinUI 3
  • Web Assembly (WASM)
  • Android
  • iOS
  • MacOS
  • Linux / GTK

Windows Build Number

  • Windows 10 1809 (Build 17763)
  • Windows 10 1903 (Build 18362)
  • Windows 10 1909 (Build 18363)
  • Windows 10 2004 (Build 19041)
  • Windows 10 20H2 (Build 19042)
  • Windows 10 21H1 (Build 19043)
  • Windows 11 21H2 (Build 22000)
  • Other (specify)

Other Windows Build number

No response

App minimum and target SDK version

  • Windows 10, version 1809 (Build 17763)
  • Windows 10, version 1903 (Build 18362)
  • Windows 10, version 1909 (Build 18363)
  • Windows 10, version 2004 (Build 19041)
  • Other (specify)

Other SDK version

No response

Visual Studio Version

No response

Visual Studio Build Number

No response

Device form factor

No response

Additional context

No response

Help us help you

Yes, I'd like to be assigned to work on this item.

Activity

  1. self-assigned this
    on Sep 16, 2024
  2. moved this to πŸ— In progress in Toolkit 8.xon Sep 16, 2024
  3. Arlodotexe commented on Sep 16, 2024

    @Arlodotexe
    MemberAuthor

    The options pane metadata is generating fine, seems the dependency property binding for the display is just failing:
    image

  4. Arlodotexe commented on Sep 16, 2024

    @Arlodotexe
    MemberAuthor

    The dependency property in GeneratedSampleOptionsRenderer.xaml.cs#L38 seems to be the root of the trouble, though it's not clear what is doing this.

    Updating the property to use an empty list by default (instead of null), there's no change in behavior. Not sure. πŸ€”

  5. Arlodotexe commented on Sep 16, 2024

    @Arlodotexe
    MemberAuthor

    I was able to fix the issue by changing the dependency property to a List<T> instead of IEnumerable<T>. Will get a PR in.

    image

  6. michael-hawker commented on Sep 16, 2024

    @michael-hawker
    Member

    Huh, odd that IEnumerable is a problem over List, I wonder why that would be...

  7. moved this from πŸ— In progress to βœ… Done in Toolkit 8.xon Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions