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

Daniel d'Andrada (dandrader) 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...

+1

« Back to merge proposal