[DataGrid] Fix scroll jump when holding down arrow keys

[arminmeh]

Fixes #15158

Reverts #14888 (which tried to fix another regression)

Scrollbar scroll positions are not set directly anymore as they are not the same number (heights of scroller and scrollbar do not match)
With smaller offsets from the top the difference is minimal, but it grows over time which is why the issue can be observed after a bit of scrolling.
The original code also impacts the performance as the values are recalculated a lot more than before the change. All this lead me to revert everything back to the state prior to #14877.

This brought back the very first issue in the chain.

I found out that it is actually caused because of a stale value in lastPosition.
Following steps in #14830 sets the lock in onScrollbarScroll, which prevents any code being executed in onScrollerScroll so the lastPosition keeps the old value. Calling scrollToIndexes end up eventually in onScrollerScroll, but it does not see the position changed, preventing the scrollbar position change as well.

So, after all these updates, the only actual change is moving up the lastPosition.current = scroller[propertyScroll]; assignment.

Internal scrollbar references are still there (comparing to the code prior to all changes), but they could be useful in future. If there are objections, I can remove them as well.

Following issues should not appear anymore
#15158
#14877 (comment)
#14830

Before: https://codesandbox.io/p/sandbox/scroll-jump-before-2fdm8g
After: https://codesandbox.io/p/sandbox/scroll-jump-after-jvv95x

[mui-bot]

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

Generated by 🚫 dangerJS against 47ae82c

[KenanYusuf]

Thanks for the context, changes lgtm.