[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).
Hmm, I can't find the comment I just made a few minutes ago about why this could happen, but I now think that comment is probably wrong anyway :) We detach our event listeners as soon as we hear the FindBar is closing, so you're right, we shouldn't need to do these null checks. I think I just added them initially since I was replacing code that was calling $() methods, which would fail silently rather than crashing if the modal bar didn't exist. Do you think there's much harm in leaving them in, though?