Yup; I'll add a TODO:. The idea is that in a follow-on CL, the Ace session management would be done by the AceEditor subclass, and this explicit setSession() call won't need to be exposed. This is essentially a temporary artifact.