[DataGrid] Fix scrollbar position not being updated after scrollToIndexes

[arminmeh]

Reverts #14877, because of a regression

Fixes #14830

Instead of changing the event listeners, new approach is to directly update scroll positions of the scrollbars together with the position of the scroller via refs

[mui-bot]

Deploy preview: https://deploy-preview-14888--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against 4724c6f

[JCQuintas]

Not sure you fixed it already, but wrapping the events in act seem to work on my side, I had encountered something similar here 🙃

    const virtualScroller = container.querySelector('.MuiDataGrid-virtualScroller')!;

    await act(async () => {
      // arbitrary number to make sure that the bottom of the grid window is reached.
      virtualScroller.scrollTop = 12345;
      virtualScroller.dispatchEvent(new Event('scroll'));
    });

    await waitFor(() => {
      expect(handleRowsScrollEnd.callCount).to.equal(1);
    });
    await act(async () => {
      setProps({
        rows: baseRows.concat(
          { id: 6, brand: 'Gucci' },
          { id: 7, brand: "Levi's" },
          { id: 8, brand: 'Ray-Ban' },
        ),
      });
      // Trigger a scroll again to notify the grid that we're not in the bottom area anymore
      virtualScroller.dispatchEvent(new Event('scroll'));
    });

    expect(handleRowsScrollEnd.callCount).to.equal(1);

    await act(async () => {
      virtualScroller.scrollTop = 12345;
      virtualScroller.dispatchEvent(new Event('scroll'));
    });

    await waitFor(() => {
      expect(handleRowsScrollEnd.callCount).to.equal(2);
    });

[romgrk]

Do these need to be public? I'd rather avoid exposing stuff until it needs to be exposed, once it's public it can't be refactored/removed.

[romgrk]

I also wanted to refactor the names to remove the "virtual" prefix from everywhere (but I couldn't because it would be a breaking change), it would make for more readable names and shorter lines. It could be interesting to name those just scrollbarXxxxRef.

[cherniavskii]

They're defined in the GridCorePrivateApi interface, so not public 👍🏻

[arminmeh]

The only public thing around this are few classes
https://mui.com/x/api/data-grid/data-grid/#data-grid-classes-MuiDataGrid-virtualScroller

we can make the change in v8 if you want to have a cleanup

[arminmeh]

by the way, merging this to fix the regression for the release
renames can be handled separately