Now that I look more closely at the interplay of objects, I'm not sure we actually need a separate `FxaModuleNavigation` class, it might be simpler to just put the navigation logic inside this object?