[replace-across-files] Unify Find Bars across FindReplace/FindInFiles

This gets rid of a lot of the redundant find bar code across single-file and multi-file find. There are still a few bits in FindInFiles that directly monkeypatch into the FindBar's ModalBar (see the TODOs in FindInFiles), but I think we can clean that up later if we want.

The first two commits are fairly semantic, and it's probably worth looking at them individually (the big one is the second one) - I added some comments in the individual commits, but if you want I can move them to the main PR diffs.

Note that this PR is a PR into the `nj/replace-across-files` branch, so this can be merged when it's reviewed (it'll just go into that long-running branch).
(We could redefine ModalBar's close event to only fire once the bar has actually been removed from the DOM. But I think we deliberately wanted everyone to forget about it as soon as the animation starts.)

Perhaps the right thing to do would be to put logic in FindBar not to trigger any events if it's already closing. @peterflynn / @redmunds What do you think - would that be safer?