[Peek] Connect BitmapImage to live XAML tree before setting source.

[gokcekantarci]

Summary of the Pull Request

https://learn.microsoft.com/en-us/windows/uwp/debug-test-perf/optimize-animations-and-media

As suggested under the "Right-sized decoding" heading, I first attached the live tree and then set source. The bug appeared to be fixed.

Note: I also tested with QOI and SVG images. I did not implement this change because I did not see any problems with these. If a similar problem occurs in a different issue, a similar solution can be applied.

PR Checklist

• [ ] Closes: #28192
• [ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
• [ ] Tests: Added/updated and all pass
• [ ] Localization: All end user facing strings can be localized
• [ ] Dev docs: Added/updated
• [ ] New binaries: Added on the required places
  • [ ] JSON for signing for new binaries
  • [ ] WXS for installer for new binaries and localization folder
  • [ ] YML for CI pipeline for new test projects
  • [ ] YML for signed pipeline
• [ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

The image above can be tested with the latest version and these changes to observe the differences.

[Jay-o-Way]

@dfederm we have a lot of problems with PRs giving errors, mentioning something with cache. Do you know why?

[dfederm]

@Jay-o-Way I'm looking into it right now. It only appears to be affecting pipelines runs from forks.

[dfederm]

/azp run

[azure-pipelines[bot]]

Azure Pipelines successfully started running 1 pipeline(s).

[dfederm]

@Jay-o-Way pipelines should be fixed with #32676. I'll monitor it today to make sure though.

Sorry for the disruption!

[Jay-o-Way]

Thanks!