Code review comment for lp:~alan-griffiths/mir/introducing-SurfaceObserver

Approve a part from:

the name SurfaceEventSource is a bit misleading... on second read: was a but misleading. Now I am only curious whether we need it inside the public headers. Would someone replacing the scene or extending the surfaces need that implementation? Or would decorating the surface factory be sufficient?