Regression in 11.1 Beta2 and RC1 concerning EmbeddableControlRoot's EmbeddableWindow implementation on Windows about avalonia

[maxkatz6]

This line is responsible for attaching embeddable control to the parent: https://github.com/AvaloniaUI/Avalonia/blob/master/src/Windows/Avalonia.Win32.Interoperability/WinForms/WinFormsAvaloniaControlHost.cs#L60

It never really did set CHILD style on parent though, so regression must come from something else. On other hand, this style would make sense anyway.

[timunie]

@fitdev a PR is welcome if you know how to improve it.

[fitdev]

It never really did set CHILD style on parent though, so regression must come from something else. On other hand, this style would make sense anyway.

I know, in fact I was using own implementation of WinFormsAvaloniaControlHost, so it is not WinFormsAvaloniaControlHost that is the culprit.

The real cause is that the "embedded window" created by EmbeddableControlRoot's constructor through IWindowingPlatform.CreateEmbeddableWindow() implemented in Win32Platform.CreateEmbeddableWindow via EmbeddedWindowImpl does not set correct styles, as expected by Windows OS for the embedding use case.

I am not Windows expert by any means, so don't really know which styles and/or extended styles should be set for such a case. But, WS_CHILD should certainly be one of them and solves this issue. Strange: it seems the style does get set in the EmbeddedWindowImpl.CreateWindowOverride, but for some reason it does not translate into actual window instance having that style in either Beta 2 or RC1.

[fitdev]

After some more digging, the culprit seems in the WindowImpl.UpdateWindowProperties which unconditionally does this WindowStyles style = WindowStyles.WS_CLIPCHILDREN | WindowStyles.WS_OVERLAPPEDWINDOW | WindowStyles.WS_CLIPSIBLINGS; and then modifies that style, without taking into account the already existing style (i.e. throws away WS_CHILD that was set by EmbeddedWindowImpl.CreateWindowOverride.

The WindowImpl.UpdateWindowProperties itself is called through:
Win32Platform.CreateEmbeddableWindow
WindowImpl.Show
WindowImpl.ShowWindow

So, the bottom line is that this gets screwed up in the WindowImpl.UpdateWindowProperties routine which does not preserve the initially correctly set WS_CHILD flag. So someone who owns this area should look into how best to pass the information to the UpdateWindowProperties that it should preserve certain window styles (perhaps via WindowProperties struct, a field, a parameter, or a check within the routine itself)

[fitdev]

Submitted PR. Can someone review it please? @timunie , @maxkatz6