Merge pull request #20433 from unoplatform/mergify/bp/release/stable/6.0/pr-20388

jeromelaban · web-flow · commit b3e6b0d376d0 · 2025-05-16T08:37:06.000-04:00
fix(webview): avoid flickering on webview initialization on win32 (backport #20388)
diff --git a/src/Uno.UI.Runtime.Skia.Win32/Win32NativeWebView.cs b/src/Uno.UI.Runtime.Skia.Win32/Win32NativeWebView.cs
@@ -100,7 +100,7 @@ public Win32NativeWebView(CoreWebView2 owner, ContentPresenter presenter)
 		}
 		_webViewForNextCreateWindow = null;
 
-		_ = PInvoke.ShowWindow(_hwnd, SHOW_WINDOW_CMD.SW_SHOWNORMAL);
+		_ = PInvoke.ShowWindow(_hwnd, SHOW_WINDOW_CMD.SW_MINIMIZE);
 
 		if (_hwnd == HWND.Null)
 		{
@@ -112,8 +112,6 @@ public Win32NativeWebView(CoreWebView2 owner, ContentPresenter presenter)
 			this.Log().Trace("Created web view window.");
 		}
 
-		Win32Host.RegisterWindow(_hwnd);
-
 		var success = PInvoke.RegisterTouchWindow(_hwnd, 0);
 		if (!success && this.Log().IsEnabled(LogLevel.Error))
 		{
@@ -182,8 +180,6 @@ private EventHandler<T> EventHandlerBuilder<T>(Action<Win32NativeWebView, object
 			{
 				_hwndToWebView.Remove(_hwnd);
 			}
-
-			Win32Host.UnregisterWindow(_hwnd);
 		});
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,7 @@ public Win32NativeWebView(CoreWebView2 owner, ContentPresenter presenter)`
`100`	`100`	`}`
`101`	`101`	`_webViewForNextCreateWindow = null;`
`102`	`102`
`103`		`- _ = PInvoke.ShowWindow(_hwnd, SHOW_WINDOW_CMD.SW_SHOWNORMAL);`
	`103`	`+ _ = PInvoke.ShowWindow(_hwnd, SHOW_WINDOW_CMD.SW_MINIMIZE);`
`104`	`104`
`105`	`105`	`if (_hwnd == HWND.Null)`
`106`	`106`	`{`
`@@ -112,8 +112,6 @@ public Win32NativeWebView(CoreWebView2 owner, ContentPresenter presenter)`
`112`	`112`	`this.Log().Trace("Created web view window.");`
`113`	`113`	`}`
`114`	`114`
`115`		`- Win32Host.RegisterWindow(_hwnd);`
`116`		`-`
`117`	`115`	`var success = PInvoke.RegisterTouchWindow(_hwnd, 0);`
`118`	`116`	`if (!success && this.Log().IsEnabled(LogLevel.Error))`
`119`	`117`	`{`
`@@ -182,8 +180,6 @@ private EventHandler<T> EventHandlerBuilder<T>(Action<Win32NativeWebView, object`
`182`	`180`	`{`
`183`	`181`	`_hwndToWebView.Remove(_hwnd);`
`184`	`182`	`}`
`185`		`-`
`186`		`- Win32Host.UnregisterWindow(_hwnd);`
`187`	`183`	`});`
`188`	`184`	`}`
`189`	`185`