fix: run the class name generator only for loaded rows

[vursen]

Description

The PR prevents the grid css class name generator from running on the rows that the data provider is still loading. That way the Flow Grid class name generator does no longer receive the cells that don't have _item assigned yet and thus the original error reported in the issue no longer takes place:

Cannot read properties of undefined (reading 'style')

Scenario: When a column group changes its hidden state, it triggers the cells' class names to re-generate. When that coincides with the data provider loading more rows, there may occur a race condition because of using debouncers so that the generator will be invoked on the rows that the virtualizer has already created but which don't have _item assigned yet.

Note, it is fine to just filter out such loading rows before running the generator since the data provider will run the generator as soon as all the rows are loaded anyway.

Fixes vaadin/flow-components#3082

Type of change

• Bugfix

Checklist

• I have read the contribution guide: https://vaadin.com/docs-beta/latest/guide/contributing/overview/
• I have added a description following the guideline.
• The issue is created in the corresponding repository and I have referenced it.
• I have added tests to ensure my change is effective and works as intended.
• New and existing tests are passing locally with my change.
• I have performed self-review and corrected misspellings.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[tomivirkki]

The change is legit, verified that it also fixes the original issue.