From: DongHyun Song <dh81.song@samsung.com>
Date: Fri, 6 Nov 2020 00:48:12 +0000 (+0900)
Subject: [Service] Terminate worker more gracefully
X-Git-Tag: submit/tizen/20201124.112151~3^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=26ba0912aac78eb121373cfe30cd803c96093fd3;p=platform%2Fframework%2Fweb%2Fwrtjs.git

[Service] Terminate worker more gracefully

process.exit() in node worker will call Worker::Exit(), which stop
the thread asyncronously. So WorkerThreadData is released later
after worker's exit event.

worker.terminate() in main thread will call Worker::StopThread(),
which processes Worker::JoinThread() as well as Worker::Exit().
JoinThread() can defer parent's exit() call after worker's gone.

Change-Id: Iad6bd2d64f158e7d6734500a7ab0623d75ad630f
Signed-off-by: DongHyun Song <dh81.song@samsung.com>
(cherry picked from commit 480b6888ff5a0e9fd5983788fc4a98d97e32e217)
---

diff --git a/wrt_app/common/service_manager.ts b/wrt_app/common/service_manager.ts
index b48f2d4..a6ab24a 100644
--- a/wrt_app/common/service_manager.ts
+++ b/wrt_app/common/service_manager.ts
@@ -25,6 +25,11 @@ function createWorker(id: string, startService: string, filename: string) {
       filename
     }
   });
+  workers[id].on('message', (message: string) => {
+    if (message === 'will-terminate') {
+      workers[id].terminate();
+    }
+  });
   workers[id].on('exit', (code: number) => {
     delete workers[id];
     let runningServices = Object.keys(workers).length;
diff --git a/wrt_app/common/service_runner.ts b/wrt_app/common/service_runner.ts
index c3c8306..3e4b39e 100644
--- a/wrt_app/common/service_runner.ts
+++ b/wrt_app/common/service_runner.ts
@@ -102,7 +102,7 @@ function run() {
       stop(id);
       setTimeout(() => {
         XWalkExtension.cleanup();
-        process.exit()
+        parentPort?.postMessage("will-terminate");
       }, message.delay);
     }
   });