[TIMOB-26662] Android: TableViewRow bugs

[drauggres]

JIRA: https://jira.appcelerator.org/browse/TIMOB-26662

This PR fixes following bugs in TableViewRow:

1. TableViewRow.getRect() always returns zeroes
2. TableViewRow height does not grow beyond original size (but can shrink)
3. NPE in TableViewRowProxy

java.lang.NullPointerException: Attempt to invoke virtual method 'ti.modules.titanium.ui.widget.TiUITableView ti.modules.titanium.ui.TableViewProxy.getTableView()' on a null object reference
    at ti.modules.titanium.ui.TableViewRowProxy.handleMessage(TableViewRowProxy.java:209)
    at android.os.Handler.dispatchMessage(Handler.java:98)
    at android.os.Looper.loop(Looper.java:154)
    at android.app.ActivityThread.main(ActivityThread.java:6144)
    at java.lang.reflect.Method.invoke(Native Method)
    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:886)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:776)```

[drauggres]

Related to #4137 (TIMOB-12658)

[build]

	Messages
📖	💾 Here's the generated SDK zipfile.
📖	🎉 Another contribution from our awesome community member, drauggres! 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.
📖	✅ All tests are passing Nice one! All 6551 tests are passing. (There are 701 skipped tests not included in that total)

Generated by 🚫 dangerJS against c896486

[drauggres]

Commits are rebased on master and reasonably squashed.
FYI: this was caused by /* global Ti */ in addontest-file (that wasn't clear at that time):

/Users/build/jenkins/workspace/ium-sdk_titanium_mobile_PR-10542/tests/Resources/ti.ui.tableview.addontest.js
8:11 error 'Ti' is already defined as a built-in global variable no-redeclare

[garymathews]

CR: PASS

[jquick-axway]

CR: Pass

[ssjsamir]

@drauggres When testing this PR against the following test case TableViewRow height does not grow beyond the original size but it seems like TableViewRow.getRect() return the correct value.

win = Ti.UI.createWindow({
backgroundColor: '#AAAAFF',
layout: 'vertical'
});
 
var top = Ti.UI.createView({
backgroundColor: '#FFAAAA',
layout: 'horizontal',
height: Ti.UI.SIZE,
width: Ti.UI.FILL
});
 
var btnPlus = Ti.UI.createButton({title: '+'});
btnPlus.addEventListener('click', function() {
console.log('plus click');
view.height += 50;
});
 
var btnMinus = Ti.UI.createButton({title: '-'});
btnMinus.addEventListener('click', function() {
console.log('minus click');
view.height -= 50;
});
 
var label = Ti.UI.createLabel();
 
var tableView = Ti.UI.createTableView();
 
var row = Ti.UI.createTableViewRow({
height: Ti.UI.SIZE,
width: Ti.UI.FILL
});
 
var view = Ti.UI.createView({
height: 150,
backgroundColor: 'blue'
});
 
row.add(view);
tableView.setData([ row ]);
tableView.addEventListener('postlayout', function onPostLayout() {
console.log('postlayout', row.rect.height, view.rect.height);
label.text = [row.rect.height, view.rect.height].join(' \| ');
});
 
top.add(btnPlus);
top.add(btnMinus);
top.add(label);
 
win.add(top);
win.add(tableView);
win.open();

Test Environment

MacOS Catalina 10.15.1 beta
Node.js 10.16.3
"NPM":"4.2.15-1","CLI":"7.1.2"
Google Pixel 7.1.1 Emulator

[garymathews]

@ssjsamir Updated PR

[ssjsamir]

FR Passed: Using the test case mentioned above.

**Test environment*Test Environment

MacOS Catalina: 10.15.1 beta
Xcode: 11.3
Java Version: 1.8.0_131
Android NDK: 18.1.5063045
Node.js: 10.16.3
"NPM":"4.2.15-1","CLI":"7.1.2-7"
Pixel xl (7.1.1) Emulator