Call revealLastElement within runAtThisOrScheduleAtNextAnimationFrame in repl

[jeanp413]

Fixes #70331

[isidorn]

Thanks for providing the PR. Added comments in the code

[isidorn]

I do not really like this call here.
When the repl gets layouted if the last element is visible we should reveal it?
Why should we reveal it if it is already visible?
Also on layout seems like a wrong place to call this. Layout happens every time the dimensions of the repl change and on the first render.

[jeanp413]

I put that extra call to reveal the last element when the inputbox gets resized because otherwise it would be covered. Also when resizing the windows.

[isidorn]

I think this approach is not good, bottom line it just calls reveal more often and with a timeout.
I believe the actual issue is in the provisional measuring of the height of the repl element.
https://github.com/Microsoft/vscode/blob/5ef80a9acde66c587a1de57a0868ac5cd24e6f1b/src/vs/workbench/contrib/debug/browser/repl.ts#L749

Note the height strategy we use with the tree which the repl uses. Repl gives the provisional height of each element so the tree would get an idea of the scroll size, and once the elments are in the dom the tree measuers them actually and adjusts.
So I think the provisional height computation should be improved and that is the root cause of this issue.

Thanks again for jumping on this issue!

[jeanp413]

Updated the PR with a better approximation for provisional height

[isidorn]

Great work, merging in.
Thanks a lot!