Missing ItemPropertyChangedEvent in GridPro

[tomivirkki]

Description

In certain rare cases, an ItemPropertyChangedEvent may not get dispatched at all (due to this check in the web component):

Kapture.2024-12-20.at.15.42.53.mp4

Related #6978

Expected outcome

The ItemPropertyChangedEvent should dispatch on editor change

Minimal reproducible example

@Route("vaadin-grid-pro/test")
public class GridProTestPage extends Div {
    public GridProTestPage() {
        GridPro<Bug> grid = new GridPro<>();

        List<Bug> bugs = List.of(new Bug("Bug 1", Severity.HIGH),
                new Bug("Bug 2", Severity.MEDIUM),
                new Bug("Bug 3", Severity.LOW));

        grid.setItems(bugs);

        Div log = new Div();

        grid.addColumn(Bug::getName).setHeader("Name");

        ComboBox<Severity> cb = new ComboBox<>();
        cb.setItems(Severity.values());
        cb.setItemLabelGenerator(severity -> severity.name());

        grid.addEditColumn(Bug::getSeverity).custom(cb, (item, newValue) -> {
            item.setSeverity(newValue);
            log.setText("Severity changed: " + newValue);
        }).setHeader("Severity");

        add(grid, log);
    }

    public static class Bug {
        private String name;
        private Severity severity;

        public Bug(String name, Severity severity) {
            this.name = name;
            this.severity = severity;
        }

        public void setName(String name) {
            this.name = name;
        }

        public String getName() {
            return name;
        }

        public void setSeverity(Severity severity) {
            this.severity = severity;
        }

        public Severity getSeverity() {
            return severity;
        }
    }

    public enum Severity {
        HIGH("3"), MEDIUM("2"), LOW("1");

        private String stringRepresentation;

        private Severity(String stringRepresentation) {
            this.stringRepresentation = stringRepresentation;
        }

        public String getStringRepresentation() {
            return stringRepresentation;
        }

        @Override
        public String toString() {
            return getStringRepresentation();
        }
    }
}

Steps to reproduce

• Open the view
• Try changing the severity from medium to high

Environment

Vaadin version(s): 24

Browsers

Issue is not browser related