I recall focus being an attribute was kind of a convenient hack to fit the existing code. Really I still don't think it should be an attribute and thus should not have a getter. Surely it's more useful to ask Mir which surface has focus rather than probe each one. You can do more with such a function.
As for visibility, AFAIK we have no client API to explicitly set visibility so it shouldn't be gettable yet either.
I recall focus being an attribute was kind of a convenient hack to fit the existing code. Really I still don't think it should be an attribute and thus should not have a getter. Surely it's more useful to ask Mir which surface has focus rather than probe each one. You can do more with such a function.
As for visibility, AFAIK we have no client API to explicitly set visibility so it shouldn't be gettable yet either.