Code review comment for lp:~osomon/webbrowser-app/shouldCancelRequests-handle-errors

Revision history for this message
Ugo Riboni (uriboni) wrote :

Complicated issue, but from the way I understand it the solution seems to make sense.
Assuming that all errors happen, if they do happen, before 500ms sounds safe enough, as these things go.
Jenkins is also happy, therefore approving, as I can't think of cases when this will make things worse than they already are anyway.

review: Approve

« Back to merge proposal