Code review comment for lp:~vanvugt/mir/surface-states

Re:
    virtual void set_event_sink(std::weak_ptr<EventSink> const&) {}

I know it's an interface so usually consists of pure virtual methods. But there's already a concrete inline destructor:
    virtual ~Session() {}

It's not that much different.