fix(ios): delay release engine resource to avoid deadlock (#4090)

wwwcg · wwwcg · commit 2d070054d069 · 2024-10-24T11:15:27.000+08:00
diff --git a/framework/ios/base/executors/HippyJSExecutor.mm b/framework/ios/base/executors/HippyJSExecutor.mm
@@ -249,7 +249,12 @@ - (void)invalidate {
         return;
     }
     dispatch_async(dispatch_get_main_queue(), ^{
+        auto engineRsc = [[HippyJSEnginesMapper defaultInstance] JSEngineResourceForKey:enginekey];
         [[HippyJSEnginesMapper defaultInstance] removeEngineResourceForKey:enginekey];
+        dispatch_async(dispatch_get_main_queue(), ^{
+            // Make a tiny delay to ensure the engine resource is released on the main thread
+            HippyLogInfo(@"Remove EngineRsc, UseCount:%ld", engineRsc.use_count());
+        });
     });
 }
 

Original file line number	Diff line number	Diff line change
`@@ -249,7 +249,12 @@ - (void)invalidate {`
`249`	`249`	`return;`
`250`	`250`	`}`
`251`	`251`	`dispatch_async(dispatch_get_main_queue(), ^{`
	`252`	`+ auto engineRsc = [[HippyJSEnginesMapper defaultInstance] JSEngineResourceForKey:enginekey];`
`252`	`253`	`[[HippyJSEnginesMapper defaultInstance] removeEngineResourceForKey:enginekey];`
	`254`	`+ dispatch_async(dispatch_get_main_queue(), ^{`
	`255`	`+ // Make a tiny delay to ensure the engine resource is released on the main thread`
	`256`	`+ HippyLogInfo(@"Remove EngineRsc, UseCount:%ld", engineRsc.use_count());`
	`257`	`+ });`
`253`	`258`	`});`
`254`	`259`	`}`
`255`	`260`