Skip to content

Conversation

oguzhankoral
Copy link
Member

TL;DR > Moved from WebView2 to CefSharp.Wpf.NETCore for Revit 2025

Reviewer should test too!

  • Open Revit 2025
  • Open relatively big model
  • Send model
  • TRY to navigate viewer of Revit -> Expectation is not frozen Revit UI and responsive UI interactions

Since the thread management is different between WebView2 and CefSharp, Revit UI was freezing in 2025 bc we moved to WebView2 with it. The source of the problem was we were running SendOperation with RevitTask.RunAsync to prevent random AccessMemoryVialoation exceptions. And running the operation with RevitTask.RunAsync was freezing Revit UI with WebView2.

Thanks to @JR-Morgan and @didimitrie for live fixing effort!

@oguzhankoral oguzhankoral marked this pull request as ready for review August 27, 2024 16:20
Copy link

codecov bot commented Aug 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 8.28%. Comparing base (93f2df0) to head (02c5926).

Additional details and impacted files
@@          Coverage Diff          @@
##             dev    #204   +/-   ##
=====================================
  Coverage   8.28%   8.28%           
=====================================
  Files        239     239           
  Lines       4683    4683           
  Branches     537     537           
=====================================
  Hits         388     388           
  Misses      4279    4279           
  Partials      16      16           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@didimitrie didimitrie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

functionally tested, LGTM

@oguzhankoral oguzhankoral merged commit 2d345e6 into dev Aug 28, 2024
5 checks passed
@oguzhankoral oguzhankoral deleted the oguzhan/revit-2025-cef-119 branch August 28, 2024 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants