From: DongHyun Song <dh81.song@samsung.com>
Date: Tue, 13 Oct 2020 08:14:25 +0000 (+0000)
Subject: Revert "[Service] Refactor stop service sequence"
X-Git-Tag: submit/tizen/20201027.083557~10
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=12ff1412f2c81ffce556a1ad5756d42996434898;p=platform%2Fframework%2Fweb%2Fwrtjs.git

Revert "[Service] Refactor stop service sequence"

This reverts commit dd79b18738cdb66f003a5195e78b5ff03248e27e.

Change-Id: If6ff1a847e9aeed86faf9fd9fc014971c339096f
---

diff --git a/wrt_app/common/service_manager.ts b/wrt_app/common/service_manager.ts
index beb3d6ea..c4b29401 100644
--- a/wrt_app/common/service_manager.ts
+++ b/wrt_app/common/service_manager.ts
@@ -43,13 +43,7 @@ export function startService(id: string, filename: string) {
   console.log(`startService - ${id}`);
   if (isMainThread) {
     let startService = `${__dirname}/service_runner.js`;
-    let worker = createWorker(id, startService, filename);
-    worker.on('message', (message: any) => {
-      if (message.type === 'stop') {
-        console.log(`${id} will shutdown after ${message.delay}ms`);
-        setTimeout(() => wrt.stopService(id), message.delay);
-      }
-    });
+    createWorker(id, startService, filename);
   }
 }
 
@@ -70,11 +64,8 @@ export function handleBuiltinService(serviceId: string, serviceName: string) {
       console.log(`Builtin service is ${serviceName}`);
       let startService = `${__dirname}/../service/builtins/${serviceName}.js`;
       let worker = createWorker(serviceId, startService, '');
-      worker.on('message', (message: any) => {
-        if (message.type === 'stop') {
-          console.log(`${serviceName} built-in service will be stopped`);
-          terminateWorker(serviceId, message.delay);
-        }
+      worker.on('stop', () => {
+        terminateWorker(serviceId, 0);
       });
     }
   }
diff --git a/wrt_app/common/service_runner.ts b/wrt_app/common/service_runner.ts
index c86829cb..43681b55 100644
--- a/wrt_app/common/service_runner.ts
+++ b/wrt_app/common/service_runner.ts
@@ -31,18 +31,13 @@ function registerExtensionResolver(id: string) {
   }
 }
 
-function requestStopService(delay: number) {
-  if (parentPort) {
-    parentPort.postMessage({type: 'stop', delay: delay});
-  }
-}
-
 let app: any = null;
 export function start(id: string, filename: string) {
   XWalkExtension.initialize();
   XWalkExtension.setRuntimeMessageHandler((type, data) => {
     if (type === 'tizen://exit') {
-      requestStopService(500);
+      console.log(`${id} will be closed by ${type}`);
+      setTimeout(() => wrt.stopService(id), 500);
     }
   });
 
@@ -71,7 +66,7 @@ export function start(id: string, filename: string) {
     }
   } catch (e) {
     console.log(`exception on start: ${e}`);
-    requestStopService(500);
+    setTimeout(() => wrt.stopService(id), 500);
   }
 }
 
diff --git a/wrt_app/service/builtins/wasm_builder.ts b/wrt_app/service/builtins/wasm_builder.ts
index 6f6c5b10..7ec30a3d 100644
--- a/wrt_app/service/builtins/wasm_builder.ts
+++ b/wrt_app/service/builtins/wasm_builder.ts
@@ -26,7 +26,7 @@ export function run(app_id: string) {
     compileWasmForCaching(file_path);
   });
   if (parentPort) {
-    parentPort.postMessage({type: 'stop', delay: 0});
+    parentPort.postMessage('stop');
   }
 }