Code review comment for lp:~alan-griffiths/mir/fix-Surface-hierarchy-make-SessionContainer-private

overall looks okay. I agree with "It and much of the code that uses it (default_session_container.cpp, global_event_sender.cpp, mediating_display_changer.cpp and session_manager.cpp) probably belong in surfaces. "

the_for_all_sessions_force_requests_to_complete_functor is a funny name... the_force_all_requests_to_complete_functor() was the only name I could come up with....