fix(android): fix table view row width ti.ui.size height

[ypbnv]

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

Description:
Fixes TableView rows with height set to Ti.UI.SIZE not returning their 'rect' properly and not
respecting children resizing.

I see the iOS is failing the unit tests. Would this behavior be expected on that platform?

Test case:
app.js

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();

[build]

• ⚠️ tests/Resources/ti.ui.tableview.addontest.js line 35 – A space is required after '['. (array-bracket-spacing)
• ⚠️ tests/Resources/ti.ui.tableview.addontest.js line 35 – A space is required before ']'. (array-bracket-spacing)

[build]

	Fails
🚫	Tests have failed, see below for more information.

	Messages
📖	💾 Here's the generated SDK zipfile.
📖	✊ 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 700 skipped out of 7252 total tests.

Tests:

Classname	Name	Time	Error
android.emulator.Titanium.UI.Android.ProgressIndicator	dialog indeterminant - show in different windows (9)	5.593	Error: timeout of 5000ms exceeded at Titanium.<anonymous> (/ti-mocha.js:6535:53723)
android.emulator.Titanium.UI.TabGroup	add Map.View to TabGroup (9)	10.065	Error: timeout of 10000ms exceeded at Titanium.<anonymous> (/ti-mocha.js:6535:53120)
android.emulator.Titanium.UI.Window.events	postlayout event gets fired (9)	20.041	Error: timeout of 20000ms exceeded at Titanium.<anonymous> (/ti-mocha.js:6535:53723)

Generated by 🚫 dangerJS against 78fb50a

[drauggres]

Hi @ypbnv.

IIRC the actual problem with TableViewRow height was here. BTW it is the only place in the SDK where property assigned to directly to LayoutParams.

Also there was fix for NullPointerException in my PR.

[build]

• ⚠️ tests/Resources/ti.ui.tableview.addontest.js line 20 – A space is required after '['. (array-bracket-spacing)
• ⚠️ tests/Resources/ti.ui.tableview.addontest.js line 20 – A space is required before ']'. (array-bracket-spacing)

[ssjsamir]

Closing in favour of #10542