[TIMOB-25886] Refactor Ti.App._restart()

[garymathews]

• Improve Ti.App._restart() performance

NOTES

• Need to test with native Titanium modules Works
• No tests as it restarts the context

TEST CASE

var win = Ti.UI.createWindow({ backgroundColor: 'gray', layout: 'vertical' }),
    globalBtn = Ti.UI.createButton({ title: 'CHECK GLOBAL (always undefined)' }),
    restartBtn = Ti.UI.createButton({ title: 'RESTART' });

globalBtn.addEventListener('click', function () {
    alert(JSON.stringify(global.TEST));
});

restartBtn.addEventListener('click', function () {
    global.TEST = true;
    Ti.App._restart();
});

setTimeout(() => {
    win.backgroundColor = 'green';
}, 3000);

win.add([ globalBtn, restartBtn ]);
win.open();

JIRA Ticket

[build]

	Messages
📖	💾 Here's the generated SDK zipfile.

Generated by 🚫 dangerJS

[jquick-axway]

Our TiLaunchActivity.loadActivityScript() and the resolveUrl() method it calls does some extra logic when loading a JavaScript on startup. Particularly with Alloy. We might need to do a code shift.
https://github.com/appcelerator/titanium_mobile/blob/master/android/titanium/src/java/org/appcelerator/titanium/TiLaunchActivity.java#L120

[garymathews]

Ah yes, I'll test these changes with an Alloy app

[garymathews]

Update, works fine with Alloy apps

[jquick-axway]

Once we set canFinishRoot to false, then backing out of the first Ti.UI.Window in a restarted window will now show the launcher activity, right? We may need to reset canFinishRoot back to true after the restart.

Perhaps this shouldn't be an exposed public global and handle it via a method.

[garymathews]

canFinishRoot returns to true on Line 229

[sgtcoolguy]

@jquick-axway ping