This is an automated email from the ASF dual-hosted git repository. luckychen pushed a commit to branch feature/0.26.2.19-jit in repository https://gitbox.apache.org/repos/asf/incubator-weex.git
commit 92982eea6224ccefe41a5682e8e845b11e486b71 Author: YorkShen <[email protected]> AuthorDate: Thu Aug 15 17:59:07 2019 +0800 [Android] Add `isJSFrameworkInit` before calling `nativeInvokeOnSuccess` --- .../main/java/com/taobao/weex/bridge/RequestHandler.java | 14 ++++++++++++-- .../main/java/com/taobao/weex/bridge/WXBridgeManager.java | 5 ++++- 2 files changed, 16 insertions(+), 3 deletions(-) diff --git a/android/sdk/src/main/java/com/taobao/weex/bridge/RequestHandler.java b/android/sdk/src/main/java/com/taobao/weex/bridge/RequestHandler.java index f8a505e..f49ba9c 100644 --- a/android/sdk/src/main/java/com/taobao/weex/bridge/RequestHandler.java +++ b/android/sdk/src/main/java/com/taobao/weex/bridge/RequestHandler.java @@ -102,7 +102,12 @@ public class RequestHandler { WXBridgeManager.getInstance().post(new Runnable() { @Override public void run() { - nativeInvokeOnSuccess(nativeCallback, content, bundleTypeStr); + if(WXBridgeManager.getInstance().isJSFrameworkInit()) { + nativeInvokeOnSuccess(nativeCallback, content, bundleTypeStr); + } + else { + nativeInvokeOnFailed(nativeCallback); + } } }); } @@ -132,7 +137,12 @@ public class RequestHandler { WXBridgeManager.getInstance().post(new Runnable() { @Override public void run() { - nativeInvokeOnSuccess(sNativeCallback, script, bundleTypeStr); + if(WXBridgeManager.getInstance().isJSFrameworkInit()) { + nativeInvokeOnSuccess(sNativeCallback, script, bundleTypeStr); + } + else{ + nativeInvokeOnFailed(sNativeCallback); + } } }); } diff --git a/android/sdk/src/main/java/com/taobao/weex/bridge/WXBridgeManager.java b/android/sdk/src/main/java/com/taobao/weex/bridge/WXBridgeManager.java index 9dafd92..8efa104 100755 --- a/android/sdk/src/main/java/com/taobao/weex/bridge/WXBridgeManager.java +++ b/android/sdk/src/main/java/com/taobao/weex/bridge/WXBridgeManager.java @@ -29,6 +29,8 @@ import android.os.Looper; import android.os.Message; import android.support.annotation.NonNull; import android.support.annotation.Nullable; +import android.support.annotation.RestrictTo; +import android.support.annotation.RestrictTo.Scope; import android.support.annotation.UiThread; import android.support.v4.util.ArrayMap; import android.text.TextUtils; @@ -366,7 +368,8 @@ public class WXBridgeManager implements Callback, BactchExecutor { // setJSFrameworkInit and isJSFrameworkInit may use on diff thread // use volatile - private boolean isJSFrameworkInit() { + @RestrictTo(Scope.LIBRARY) + boolean isJSFrameworkInit() { return mInit; }
