[Merge chakra-core/ChakraCore@860ad1275e] [MERGE #3719 @jianchun] fix…

… JavascriptFunction::CheckValidDebugThunk CrossSite check

Merge pull request #3719 from jianchun:validdbgthunk

Comparing with scriptContext->CurrentCrossSiteThunk is unreliable. The
function being checked could be CrossSite and its context closed, thus
could be using a cross site thunk different to scriptContext's (note we
have 2 cross site thunks: DefaultThunk and ProfileThunk).

Change to CrossSite::IsThunk to cover both.

Fix OS#11699485

deps/chakrashim/core/lib/Runtime/Library/JavascriptFunction.cpp

@@ -806,7 +806,7 @@ namespace Js
         if (scriptContext->IsScriptContextInDebugMode()
             && !scriptContext->IsInterpreted() && !CONFIG_FLAG(ForceDiagnosticsMode)    // Does not work nicely if we change the default settings.
             && function->GetEntryPoint() != scriptContext->CurrentThunk
-            && function->GetEntryPoint() != scriptContext->CurrentCrossSiteThunk
+            && !CrossSite::IsThunk(function->GetEntryPoint())
             && JavascriptFunction::Is(function))
         {