fix(android): handle shortcut exception

[m1ga]

JIRA: https://jira.appcelerator.org/browse/AC-6564

Optional Description:

Won't crash with an unhandled exception:

var win = Ti.UI.createWindow({
		backgroundColor: 'grey'
	}),
	btn = Ti.UI.createButton({
		top: 0,
		title: 'TOGGLE SHORTCUT'
	})

Ti.App.addEventListener('shortcutitemclick', function() {
	win.backgroundColor = 'blue';
});

btn.addEventListener('click', function() {
	for (var i = 0; i < 10; ++i) {
		var shortcut = Ti.UI.createShortcutItem({
			id: 'test_shortcut' + i,
			title: 'TEST' + i,
			description: 'DESCRIPTION' + i
		});
		if (shortcut.visible) {
			win.backgroundColor = 'red';
			shortcut.hide();
		} else {
			win.backgroundColor = 'green';
			shortcut.show();
		}
	}
});

win.add(btn);
win.open();

[build]

	Fails
🚫	🔬 There are library changes, but no changes to the unit tests. That's OK as long as you're refactoring existing code, but will require an admin to merge this PR. Please see README.md#unit-tests for docs on unit testing.
🚫	Tests have failed, see below for more information.

	Messages
📖	💾 Here's the generated SDK zipfile.
📖	🎉 Another contribution from our awesome community member, m1ga! Thanks again for helping us make Titanium SDK better. 👍
📖	✊ The commits in this PR match our conventions! Feel free to Rebase and Merge this PR when ready.
📖	❌ 3 tests have failed There are 3 tests failing and 698 skipped out of 7345 total tests.

Tests:

Classname	Name	Time	Error
android.emulator.Titanium.Android	activity callbacks (9)	5.11	Error: timeout of 5000ms exceeded at Titanium.<anonymous> (/ti-mocha.js:6535:53120)
android.emulator.Titanium.UI.TextArea	.focused (9)	2.443	Error: timeout of 2000ms exceeded at Titanium.<anonymous> (/ti-mocha.js:6535:53723)
android.emulator.Titanium.UI.TextField	.focused (9)	2.028	Error: timeout of 2000ms exceeded at Titanium.<anonymous> (/ti-mocha.js:6535:53723)

Generated by 🚫 dangerJS against 6bfa28a

[garymathews]

I have pushed an update to this PR, however this may be superseded by #11759

[sgtcoolguy]

@garymathews What should we do with this PR now that this code has been re-written? It looks to me like a use case we still need to protect against in the refactored code. Specifically here: https://github.com/appcelerator/titanium_mobile/blob/master/android/modules/ui/src/java/ti/modules/titanium/ui/ShortcutProxy.java#L139 and here: https://github.com/appcelerator/titanium_mobile/blob/master/android/modules/ui/src/java/ti/modules/titanium/ui/ShortcutItemProxy.java#L124

[garymathews]

Closing as #11759 has already addressed this issue.