Bug 571744 - AbstractTableViewer.virtualSetSelectionToWidget(List, boolean) should use IIndexableLazyContentProvider
Summary: AbstractTableViewer.virtualSetSelectionToWidget(List, boolean) should use IIn...
Status: NEW
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.20   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Platform-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-03-06 02:12 EST by Christoph Laeubrich CLA
Modified: 2021-03-06 02:12 EST (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Christoph Laeubrich CLA 2021-03-06 02:12:01 EST 
AbstractTableViewer.virtualSetSelectionToWidget currently possibly iterates through all elements to get the index of a selection for VirtualTables.

The code contains the following comments:

// Now go through it again until all is done or we are no longer
// virtual
// This may create all items so it is not a good
// idea in general.

a "better" idea would be to at least check for IIndexableLazyContentProvider and call IIndexableLazyContentProvider.findElement(Object) instead if possible.