[TIMOB-24560] Android: Prevent duplicate TableViewRowProxyItem on Android 7.0

[garymathews]

• On Android 7.0 getView() re-uses previous TableViewRowProxyItem instances
• Prevent duplicate TableViewRowProxyItem from occuring

TEST CASE

var win = Ti.UI.createWindow({
        backgroundColor: 'white',
        layout: 'vertical'
    }),
    tableData = [{
        title: 'Apples',
        color: 'black'
    }, {
        title: 'Bananas',
        color: 'black'
    }, {
        title: 'Carrots',
        color: 'black'
    }, {
        title: 'Potatoes',
        color: 'black'
    }],
    table = Ti.UI.createTableView({
        minRowHeight: 50,
        data: tableData,
        width: Ti.UI.FILL,
        height: Ti.UI.SIZE
    });
table.addEventListener('click', function(e) {
    alert('index: ' + e.index);
});
win.add(table);
win.open();

JIRA Ticket

[jquick-axway]

I tested it and it works.

I am concerned that this will cause a performance issue with TableViews having a lot of rows (it has an O(n^2) complexity now), but we can deal with that later if it ever comes up.

[longton95]

FR passed.

Tested using the test case above, there is no longer duplicate index data and the correct index is returned.

ENV

SDK Version : Local 6.2.0
Mac OS Version : 10.12.4
Appc CLI : 6.2.1
Appc NPM : 4.2.9
Node : v6.10.3
Device: Nexus 5x(6.0.1), Pixel XL(7.1.2)
Emulator: (7.0), (6.0), (4.4.2)