[browser][MT] troubleshooting deadlocks 2 (#104615)

dotnet · Jul 10, 2024 · ed9642e · ed9642e
1 parent a2b7214
commit ed9642e
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/...ces.JavaScript/src/System/Runtime/InteropServices/JavaScript/Interop/JavaScriptExports.cs b/...ces.JavaScript/src/System/Runtime/InteropServices/JavaScript/Interop/JavaScriptExports.cs
@@ -277,7 +277,7 @@ public static void InstallMainSynchronizationContext(JSMarshalerArgument* argume
             }
             catch (Exception ex)
             {
-                arg_exc.ToJS(ex);
+                Environment.FailFast($"InstallMainSynchronizationContext: Unexpected failure (ManagedThreadId {Environment.CurrentManagedThreadId}): " + ex);
             }
         }
 

diff --git a/...ices.JavaScript/src/System/Runtime/InteropServices/JavaScript/JSSynchronizationContext.cs b/...ices.JavaScript/src/System/Runtime/InteropServices/JavaScript/JSSynchronizationContext.cs
@@ -294,9 +294,9 @@ private void Pump()
                 // otherwise, we could schedule another round
                 AwaitNewData();
             }
-            catch (Exception e)
+            catch (Exception ex)
             {
-                Environment.FailFast($"JSSynchronizationContext.Pump failed, ManagedThreadId: {Environment.CurrentManagedThreadId}. {Environment.NewLine} {e.StackTrace}");
+                Environment.FailFast($"JSSynchronizationContext.Pump: Unexpected failure (ManagedThreadId {Environment.CurrentManagedThreadId}): " + ex);
             }
         }
 

diff --git a/...pServices.JavaScript.UnitTests/System/Runtime/InteropServices/JavaScript/WebWorkerTest.cs b/...pServices.JavaScript.UnitTests/System/Runtime/InteropServices/JavaScript/WebWorkerTest.cs
@@ -520,6 +520,7 @@ await executor.Execute(async () =>
         }
 
         [Theory, MemberData(nameof(GetTargetThreadsAndBlockingCalls))]
+        [ActiveIssue("https://github.com/dotnet/runtime/issues/99951")]
         public async Task WaitAssertsOnSyncCallback(Executor executor, NamedCall method)
         {
             using var cts = CreateTestCaseTimeoutSource();
@@ -555,6 +556,7 @@ await WebWorkerTestHelper.CallMeBackSync(() =>
         }
 
         [Theory, MemberData(nameof(GetTargetThreadsAndBlockingCalls))]
+        [ActiveIssue("https://github.com/dotnet/runtime/issues/99951")]
         public async Task WaitAssertsOnSyncJSExport(Executor executor, NamedCall method)
         {
             using var cts = CreateTestCaseTimeoutSource();