Code review comment for lp:~lukas-kde/unity8/fix-shell-chrome

Revision history for this message
Michael Zanetti (mzanetti) wrote :

> > hmm... a couple of questions:
> >
> > * A timer of 1 second seems quite long. Is that really what we want?
>
> Well yes :/ try lowering the value and see it fail more and more often.

Hmm... I'm not entirely happy with that... can't we somehow trigger this as a result of the other stuff being finished instead of having a timer that is a) so slow, it's visible to the user, and b) might still fail if the machine is under load or something...

« Back to merge proposal