Xml Doc improvements

Author: amaitland

CefSharp.WinForms/ChromiumWebBrowser.cs

@@ -376,7 +376,7 @@ public IRequestContext RequestContext
         /// <summary>
         /// ParentFormMessageInterceptor hooks the Form handle and forwards
         /// the move/active messages to the browser, the default is true
-        /// and should only be required when using <see cref="CefSettings.MultiThreadedMessageLoop"/>
+        /// and should only be required when using <see cref="AbstractCefSettings.MultiThreadedMessageLoop"/>
         /// set to true.
         /// </summary>
         [Browsable(false), DesignerSerializationVisibility(DesignerSerializationVisibility.Hidden), DefaultValue(true)]
@@ -619,7 +619,7 @@ protected override void OnHandleCreated(EventArgs e)
         /// To re-enable Window Activation then remove WS_EX_NOACTIVATE from ExStyle
         /// <code>
         /// const uint WS_EX_NOACTIVATE = 0x08000000;
-        /// windowInfo.ExStyle &= ~WS_EX_NOACTIVATE;
+        /// windowInfo.ExStyle &amp;= ~WS_EX_NOACTIVATE;
         ///</code>
         /// </example>
         protected virtual IWindowInfo CreateBrowserWindowInfo(IntPtr handle)

CefSharp.Wpf/ChromiumWebBrowser.cs

@@ -36,7 +36,14 @@ namespace CefSharp.Wpf
     [TemplatePart(Name = PartPopupImageName, Type = typeof(Image))]
     public class ChromiumWebBrowser : Control, IRenderWebBrowser, IWpfWebBrowser
     {
+        /// <summary>
+        /// TemplatePart Name constant for the Image used to represent the browser
+        /// </summary>
         public const string PartImageName = "PART_image";
+        /// <summary>
+        /// TemplatePart Name constant for the Image used to represent the popup
+        /// overlayed on the browser
+        /// </summary>
         public const string PartPopupImageName = "PART_popupImage";
 
         /// <summary>
@@ -1981,6 +1988,13 @@ internal void UiThreadRunAsync(Action action, DispatcherPriority priority = Disp
             }
         }
 
+        /// <summary>
+        /// Run the Action on the CEF UI Thread in an async fashion
+        /// </summary>
+        /// <param name="action">The action.</param>
+        /// <returns>
+        /// An asynchronous result.
+        /// </returns>
         protected async Task CefUiThreadRunAsync(Action action)
         {
             if (!IsDisposed && InternalIsBrowserInitialized())
@@ -2471,6 +2485,11 @@ protected override void OnMouseLeave(MouseEventArgs e)
             base.OnMouseLeave(e);
         }
 
+        /// <summary>
+        /// Invoked when an unhandled <see cref="E:System.Windows.Input.Mouse.LostMouseCapture" /> attached event reaches an element in
+        /// its route that is derived from this class. Implement this method to add class handling for this event.
+        /// </summary>
+        /// <param name="e">The <see cref="T:System.Windows.Input.MouseEventArgs" /> that contains event data.</param>
         protected override void OnLostMouseCapture(MouseEventArgs e)
         {
             if (!e.Handled && browser != null)

CefSharp.Wpf/Experimental/WpfIMEKeyboardHandler.cs

@@ -15,6 +15,10 @@
 
 namespace CefSharp.Wpf.Experimental
 {
+    /// <summary>
+    /// A WPF Keyboard handler implementation that supports IME
+    /// </summary>
+    /// <seealso cref="T:CefSharp.Wpf.Internals.WpfKeyboardHandler"/>
     public class WpfImeKeyboardHandler : WpfKeyboardHandler
     {
         private int languageCodeId;
@@ -27,10 +31,19 @@ public class WpfImeKeyboardHandler : WpfKeyboardHandler
         private MouseButtonEventHandler mouseDownEventHandler;
         private bool isActive;
 
+        /// <summary>
+        /// Constructor.
+        /// </summary>
+        /// <param name="owner">The owner.</param>
         public WpfImeKeyboardHandler(ChromiumWebBrowser owner) : base(owner)
         {
         }
 
+        /// <summary>
+        /// Change composition range.
+        /// </summary>
+        /// <param name="selectionRange">The selection range.</param>
+        /// <param name="characterBounds">The character bounds.</param>
         public void ChangeCompositionRange(Range selectionRange, Rect[] characterBounds)
         {
             if (!isActive)
@@ -70,6 +83,10 @@ public void ChangeCompositionRange(Range selectionRange, Rect[] characterBounds)
             });
         }
 
+        /// <summary>
+        /// Setup the Ime Keyboard Handler specific hooks and events
+        /// </summary>
+        /// <param name="source">HwndSource.</param>
         public override void Setup(HwndSource source)
         {
             if (isSetup)
@@ -98,6 +115,9 @@ public override void Setup(HwndSource source)
             }
         }
 
+        /// <summary>
+        /// Performs application-defined tasks associated with freeing, releasing, or resetting unmanaged resources.
+        /// </summary>
         public override void Dispose()
         {
             // Note Setup can be run after disposing, to "reset" this instance
@@ -245,6 +265,10 @@ private void OnImeComposition(IntPtr hwnd, int lParam)
             }
         }
 
+        /// <summary>
+        /// Cancel composition.
+        /// </summary>
+        /// <param name="hwnd">The hwnd.</param>
         public void CancelComposition(IntPtr hwnd)
         {
             owner.GetBrowserHost().ImeCancelComposition();