Skip to content

Commit caa26a5

Browse files
authored
Merge eb57fa2 into 24e7df2
2 parents 24e7df2 + eb57fa2 commit caa26a5

File tree

1 file changed

+12
-2
lines changed

1 file changed

+12
-2
lines changed

CefSharp.BrowserSubprocess.Core/CefAppUnmanagedWrapper.cpp

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,7 @@ namespace CefSharp
167167

168168
auto browserWrapper = FindBrowserWrapper(browser->GetIdentifier());
169169

170-
if (browserWrapper != nullptr && browserWrapper->JavascriptBindingApiEnabled && IsJavascriptBindingApiAllowed(browserWrapper, frame))
170+
if (IsJavascriptBindingApiAllowed(browserWrapper, frame))
171171
{
172172
//TODO: Look at adding some sort of javascript mapping layer to reduce the code duplication
173173
auto global = context->GetGlobal();
@@ -349,7 +349,17 @@ namespace CefSharp
349349

350350
bool CefAppUnmanagedWrapper::IsJavascriptBindingApiAllowed(CefBrowserWrapper^ browserWrapper, CefRefPtr<CefFrame> frame)
351351
{
352-
if (browserWrapper == nullptr || !browserWrapper->JavascriptBindingApiHasAllowOrigins)
352+
if (browserWrapper == nullptr)
353+
{
354+
return true;
355+
}
356+
357+
if (!browserWrapper->JavascriptBindingApiEnabled)
358+
{
359+
return false;
360+
}
361+
362+
if (!browserWrapper->JavascriptBindingApiHasAllowOrigins)
353363
{
354364
return true;
355365
}

0 commit comments

Comments
 (0)