ObservableBoolean should have a weak reference to its observer.

This is to prevent |observer| from ever being a dangling pointer, thus making our code less crash-prone.

In addition this would allow us to remove nearly all occurrences of "setObserver:nil" in the codebase, and thus remove a bunch of deallocs.