Remove Spree::OrderObserver.

It's not clear why this observer was necessary, since there is already an observer on payment which should run after_finalize. It was introduced in 2c1ba03b38649a5b6823f125ddfbe709f33ce8bc.

If anyone has a clue why this was needed, I'd be keen to hear.