Code Push Android App crash after Update

[kumark86]

Steps to Reproduce

1. Open the app and got the update dialog from code push
2. Install the update
3. App got crash after getting updated

Expected Behavior

What you expected to happen?

Update the app from code push

Actual Behavior

What actually happens?

 com.facebook.react.uimanager.IllegalViewOperationException: Trying to remove a view index above child count 0 view tag: 11
                                                  detail: View tag:-1
                                                   children(0): [
                                                  ],
                                                   indicesToRemove(1): [
                                                 0,
                                                  ],
                                                   tagsToDelete(1): [
                                                 156,
                                                  ]
                                                 
                                                     at com.facebook.react.uimanager.NativeViewHierarchyManager.manageChildren(NativeViewHierarchyManager.java:346)
                                                     at com.facebook.react.uimanager.UIViewOperationQueue$ManageChildrenOperation.execute(UIViewOperationQueue.java:177)
                                                     at com.facebook.react.uimanager.UIViewOperationQueue$1.run(UIViewOperationQueue.java:776)
                                                     at com.facebook.react.uimanager.UIViewOperationQueue.flushPendingBatches(UIViewOperationQueue.java:855)
                                                     at com.facebook.react.uimanager.UIViewOperationQueue.access$1600(UIViewOperationQueue.java:46)
                                                     at com.facebook.react.uimanager.UIViewOperationQueue$2.runGuarded(UIViewOperationQueue.java:813)
                                                     at com.facebook.react.bridge.GuardedRunnable.run(GuardedRunnable.java:21)
                                                     at android.os.Handler.handleCallback(Handler.java:733)
                                                     at android.os.Handler.dispatchMessage(Handler.java:95)
                                                     at android.os.Looper.loop(Looper.java:136)
                                                     at android.app.ActivityThread.main(ActivityThread.java:5584)
                                                     at java.lang.reflect.Method.invokeNative(Native Method)
                                                     at java.lang.reflect.Method.invoke(Method.java:515)
                                                     at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1268)
                                                     at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1084)
                                                     at dalvik.system.NativeStart.main(Native Method)

Environment

• react-native-code-push version: 4.0.0-beta
• react-native version:0.46.1
• iOS/Android/Windows version: kitkat
• Does this reproduce on a debug build or release build? release build
• Does this reproduce on a simulator, or only on a physical device? I have tested only in physical device

(The more info the faster we will be able to address it!)

[max-mironov]

Relates to #986 issue.
@kumark86 - do you have any stable conditions where we can replicate it? Can you share a quick sample app so we can reproduce it locally?

[max-mironov]

@kumark86 do you have any updates regarding this? Are you able to reproduce it constantly?

[sood03]

@max-mironov Yes this can be constantly reproduced with the CodePush.sync() method defined as in #986

[max-mironov]

@sood03 thanks, got it! I'll try to reproduce your issue with configuration you have provided on demo app and let you know the results. BTW what is the android version you are testing? Is it android 7? Do you have any other aspects in your application that I could probably miss?

[sood03]

Hi @max-mironov , I am on RN 0.45.1 and RNCP 3.0.0-beta. I tried on Android 7 (emulator) and on Android 6.0 on Redmi Note 3 facing the same error. The same settings work perfectly fine on iOS. I am also using react-native-modalbox to show the installation progress.

This is the render method in my js which handles CodePush

import Modal from 'react-native-modalbox';


render() {
    if (this.state.showDownloadingModal) {
    return (
        <Container>
            <Content style={styles.container}>
                <Modal
                    style={[styles.modal, styles.modal1]}
                    backdrop={false}
                    ref={c => {
                    this._modal = c;
                    }}
                    swipeToClose={false}
                >
                    <View
                        style={{
                            flex: 1,
                            alignSelf: "stretch",
                            justifyContent: "center",
                        }}
                    >
                    {this.state.showInstalling
                    ? <Text
                        style={{
                            textAlign: "center",
                            marginBottom: 15,
                            fontSize: 15
                        }}
                    >
                    Installing update...
                    </Text>
                    : <View
                            style={{
                                flex: 1,
                                alignSelf: "stretch",
                                justifyContent: "center",
                                padding: 20
                            }}
                        >
                        <Text
                            style={{
                                color: 'white',
                                textAlign: "center",
                                marginBottom: 15,
                                fontSize: 15
                            }}
                        >
                        Downloading update...
                        {" "}
                        {`${parseInt(this.state.downloadProgress, 10)} %`}
                        </Text>
                        <ProgressBar
                            color="white"
                            progress={parseInt(this.state.downloadProgress, 10)}
                        />
                        </View>}
                    </View>
                </Modal>
            </Content>
        </Container>
);
}
return <MainStackRouter />;
}

[sood03]

Sorry for the formatting. Don't know why are the tabs not honored if I add it like a code snippet.

[max-mironov]

@sood03 np, you should use ``` instead of ` to keep tabs.

As for your issue - I've tried to reproduce it with RN45.1 and RNCP3.0.0 but with no luck.
I've attached archive with my demo (you'll need to npm i and change code-push deployment key), so you can try it locally to see if you will be able to reproduce the issue somehow.
Please let us know if you can find anything.
AndroidCrash.zip

[max-mironov]

@sood03 sorry to bother you but did you have a chance to try to reproduce your case with the demo app I've provided?

[sood03]

Hi @max-mironov , Sorry I couldn't get time to check this until now. I will try this today and get back to you soon.

[max-mironov]

@sood03 thanks for reply, please let us know if you could find anything.

[AndrewJack]

I think this issue might have something to do with updating state (and causing a render) on the UPDATE_INSTALLED event.
After this event the app will restart. This could cause issues if you are updating the same thing that is being destroyed.

Still need to test this theory though.

[max-mironov]

@AndrewJack thanks for reply, what do you mean by

updating the same thing that is being destroyed.

I've tried immediate update and remove visible element from the app but seems ok. Do you have more precise scenario we can try to test to see if we have an issue?