WinForms - Refactor ChromiumHostControlBase to use P/Invoke

- Avoid using using NativeMethodWrapper, hopefully this keeps the designer
  happy, no need to load CefSharp.Core.Runtime.dll

Issue #4019

Author: amaitland

CefSharp.WinForms/Host/ChromiumHostControlBase.cs

@@ -6,6 +6,7 @@
 using System.ComponentModel;
 using System.Drawing;
 using System.Runtime.CompilerServices;
+using System.Runtime.InteropServices;
 using System.Windows.Forms;
 
 namespace CefSharp.WinForms.Host
@@ -17,6 +18,10 @@ namespace CefSharp.WinForms.Host
     /// <seealso cref="System.Windows.Forms.Control" />
     public abstract class ChromiumHostControlBase : Control
     {
+        [DllImport("user32.dll", SetLastError = true)]
+        [return: MarshalAs(UnmanagedType.Bool)]
+        private static extern bool SetWindowPos(IntPtr hWnd, IntPtr hWndInsertAfter, int X, int Y, int cx, int cy, uint uFlags);
+
         /// <summary>
         /// IntPtr that represents the CefBrowser Hwnd
         /// Used for sending messages to the browser
@@ -121,25 +126,10 @@ protected virtual void ResizeBrowser(int width, int height)
         {
             if (BrowserHwnd != IntPtr.Zero)
             {
-                ResizeBrowserInternal(Width, Height);
+                SetWindowPosition(BrowserHwnd, 0, 0, width, height);
             }
         }
 
-        /// <summary>
-        /// Resizes the browser.
-        /// </summary>
-        /// <param name="width">width</param>
-        /// <param name="height">height</param>
-        /// <remarks>
-        /// To avoid the Designer trying to load CefSharp.Core.Runtime we explicitly
-        /// ask for NoInlining.
-        /// </remarks>
-        [MethodImpl(MethodImplOptions.NoInlining)]
-        private void ResizeBrowserInternal(int width, int height)
-        {
-            NativeMethodWrapper.SetWindowPosition(BrowserHwnd, 0, 0, width, height);
-        }
-
         /// <summary>
         /// When minimized set the browser window size to 0x0 to reduce resource usage.
         /// https://github.com/chromiumembedded/cef/blob/c7701b8a6168f105f2c2d6b239ce3958da3e3f13/tests/cefclient/browser/browser_window_std_win.cc#L87
@@ -148,7 +138,7 @@ internal virtual void HideInternal()
         {
             if (BrowserHwnd != IntPtr.Zero)
             {
-                NativeMethodWrapper.SetWindowPosition(BrowserHwnd, 0, 0, 0, 0);
+                SetWindowPosition(BrowserHwnd, 0, 0, 0, 0);
             }
         }
 
@@ -159,7 +149,7 @@ internal virtual void ShowInternal()
         {
             if (BrowserHwnd != IntPtr.Zero)
             {
-                NativeMethodWrapper.SetWindowPosition(BrowserHwnd, 0, 0, Width, Height);
+                SetWindowPosition(BrowserHwnd, 0, 0, Width, Height);
             }
         }
 
@@ -183,6 +173,27 @@ internal void RaiseIsBrowserInitializedChangedEvent()
             IsBrowserInitializedChanged?.Invoke(this, EventArgs.Empty);
         }
 
+        private void SetWindowPosition(IntPtr handle, int x, int y, int width, int height)
+        {
+            const uint SWP_NOMOVE = 0x0002;
+            const uint SWP_NOZORDER = 0x0004;
+            const uint SWP_NOACTIVATE = 0x0010;
+
+            if (handle != IntPtr.Zero)
+            {
+                if (width == 0 && height == 0)
+                {
+                    // For windowed browsers when the frame window is minimized set the
+                    // browser window size to 0x0 to reduce resource usage.
+                    SetWindowPos(handle, IntPtr.Zero, 0, 0, 0, 0, SWP_NOZORDER | SWP_NOMOVE | SWP_NOACTIVATE);
+                }
+                else
+                {
+                    SetWindowPos(handle, IntPtr.Zero, x, y, width, height, SWP_NOZORDER);
+                }
+            }
+        }
+
         /// <summary>
         /// Gets the <see cref="ChromiumHostControl"/> or <see cref="ChromiumWebBrowser"/> associated with
         /// a specific <see cref="IBrowser"/> instance.