Fix: Set correct hwnd for temporary webview2 controller #3169
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes proposed in this pull request
Related issue(s)
Copilot generated summary
Provide a Copilot generated summary of the changes in this pull request.
Copilot summary
This pull request updates the way the application handles the deletion of browsing data in the web console settings. The main change is that the code now passes the main window handle when creating the WebView2 controller, which improves compatibility and stability when clearing browsing data.
Improvements to WebView2 controller initialization:
DeleteBrowsingData()inWebConsoleSettingsViewModel.csto use the main window handle (WindowInteropHelper(Application.Current.MainWindow).Handle) when creating the WebView2 controller, instead of passingIntPtr.Zero. This change ensures the controller is properly associated with the application's main window.using System;directive fromWebConsoleSettingsViewModel.cs, as it is no longer needed after the change.To-Do
Contributing
By submitting this pull request, I confirm the following: