Implement Go To Next/Previous Breakpoint editor actions

[Krzysztof-Cieslak]

Fix #31806

CC: @isidorn

[isidorn]

editor Model can be null

[Krzysztof-Cieslak]

It's not really checked in any other place in the file ;-)

[isidorn]

Instead of doing a .filter on top, please pass a "enabledOnly: true" to the getBreakpoints

[isidorn]

Use openBreakpointSource instead

[isidorn]

Can we do this in twice as less lines? seems like a lot of code is duplicated in the next 20 lines?

[Krzysztof-Cieslak]

The sorting is unnecessary in the end since debugModel is responsible for sorting its own breakpoints - https://github.com/Microsoft/vscode/blob/cdd8256102433bf52c2f5b05aea312775a34a669/src/vs/workbench/parts/debug/common/debugModel.ts#L1003-L1015

[isidorn]

@Krzysztof-Cieslak thanks a lot for your PR. I have commented directly in the file.
Once you address those changes we can merge this in. Thanks!

[isidorn]

You seem to have rebased instead of merge. I suggest to create a new branch to cleanup the mess.

[Krzysztof-Cieslak]

I've figured out my small problems with git, I think it's more or less ready ;-)

[isidorn]

@Krzysztof-Cieslak thanks a lot, this looks great. Merged it in 🍻