remove cruft
refactor to work on BPAction type instead of variables
Fix race condition when reloading breakpoints

sync() was getting called before all the code panes were drawn, and so the
line elements with breakpoints did not yet exist when we pulled down
the already existing breakpoint data
remove cruft
remove items from the breakpoint list by clicking on the X
common-ize deleting breakpoints and actions