Refactor LogSink #8555

EventEmitter can be moved inside InnerLogSink, this will also make spawning a task to pass on the events in subscribe redundant. After this, timeouts inside InnerLogSink::assert and InnerLogSink::assert_warns_and_errors will no longer be needed and can be changed to try_recv calls.

Follow up to #8457.