IBrowserHost.GetVisibleNavigationEntry - Add call to ThrowIfExecutedO…

…nNonCefUiThread guard and improve xml doc Resolves #2986
cefsharp · Dec 9, 2019 · 99c177a · 99c177a
1 parent db4f5f3
commit 99c177a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CefSharp.Core/Internals/CefBrowserHostWrapper.cpp b/CefSharp.Core/Internals/CefBrowserHostWrapper.cpp
@@ -492,6 +492,8 @@ NavigationEntry^ CefBrowserHostWrapper::GetVisibleNavigationEntry()
 {
     ThrowIfDisposed();
 
+    ThrowIfExecutedOnNonCefUiThread();
+
     auto entry = _browserHost->GetVisibleNavigationEntry();
 
     return TypeConversion::FromNative(entry, true);

diff --git a/CefSharp/IBrowserHost.cs b/CefSharp/IBrowserHost.cs
@@ -396,7 +396,8 @@ public interface IBrowserHost : IDisposable
 
         /// <summary>
         /// Returns the current visible navigation entry for this browser. This method
-        /// can only be called on the CEF UI thread.
+        /// can only be called on the CEF UI thread which by default is not the same
+        /// as your application UI thread.
         /// </summary>
         /// <returns>the current navigation entry</returns>
         NavigationEntry GetVisibleNavigationEntry();