in fa mode, when JsParticleAbility connect to other service, it could cause crash with disconnect-callback invalid

Signed-off-by: wangzhen <wangzhen346@huawei.com>

frameworks/js/napi/featureAbility/feature_ability.cpp

@@ -106,11 +106,8 @@ private:
 
 void JsFeatureAbility::Finalizer(NativeEngine *engine, void *data, void *hint)
 {
-    HILOG_DEBUG("JsFeatureAbility::Finalizer is called");
-    auto pthis = std::unique_ptr<JsFeatureAbility>(static_cast<JsFeatureAbility*>(data));
-    if (pthis) {
-        pthis->RemoveAllCallbacksLocked();
-    }
+    HILOG_INFO("JsFeatureAbility::Finalizer is called");
+    std::unique_ptr<JsFeatureAbility>(static_cast<JsFeatureAbility*>(data));
 }
 
 NativeValue* JsFeatureAbilityInit(NativeEngine *engine, NativeValue *exports)

frameworks/js/napi/inner/napi_ability_common/js_napi_common_ability.h

@@ -28,7 +28,7 @@ namespace AppExecFwk {
 class JsNapiCommon {
 public:
     JsNapiCommon();
-    virtual ~JsNapiCommon() = default;
+    virtual ~JsNapiCommon();
 
     struct JsPermissionOptions {
         bool uidFlag = false;

frameworks/js/napi/inner/napi_ability_common/napi_common_ability.cpp

@@ -3982,6 +3982,11 @@ napi_value NAPI_CancelBackgroundRunningCommon(napi_env env, napi_callback_info i
 JsNapiCommon::JsNapiCommon() : ability_(nullptr)
 {}
 
+JsNapiCommon::~JsNapiCommon()
+{
+    RemoveAllCallbacksLocked();
+}
+
 NativeValue* JsNapiCommon::JsConnectAbility(
     NativeEngine &engine, NativeCallbackInfo &info, const AbilityType abilityType)
 {

frameworks/js/napi/particleAbility/particle_ability.cpp

@@ -201,7 +201,7 @@ napi_value ParticleAbilityInit(napi_env env, napi_value exports)
 
 void JsParticleAbility::Finalizer(NativeEngine *engine, void *data, void *hint)
 {
-    HILOG_INFO("JsWantAgent::Finalizer is called");
+    HILOG_INFO("JsParticleAbility::Finalizer is called");
     std::unique_ptr<JsParticleAbility>(static_cast<JsParticleAbility*>(data));
 }