Page MenuHomePhabricator
Paste P282

For the record re: sleep 1 in browser tests
ActivePublic

Authored by MarkTraceur on Feb 11 2015, 5:55 PM.
2015-02-11 - 11:47:53 <marktraceur> chrismcmahon: The Chrome bugs in our browser tests that require sleep 1 everywhere, is there any bug in upstream tracking that?
2015-02-11 - 11:50:17 <+chrismcmahon> marktraceur: afaik, the Chrome people consider it a feature not a bug
2015-02-11 - 11:51:33 <+chrismcmahon> marktraceur: and (again afaik) because of the way RL works, it is fiendishly difficult to work around
2015-02-11 - 11:52:03 <marktraceur> *sigh*
2015-02-11 - 11:52:13 <marktraceur> chrismcmahon: Should we just bite the bullet and add the sleep 1 everywhere then?
2015-02-11 - 11:52:34 <+chrismcmahon> marktraceur: I say yeah, sleep 1 is not that evil. sleep 10 would be a lot worse
2015-02-11 - 11:52:46 <marktraceur> OK
2015-02-11 - 11:52:58 * marktraceur will !logs this conversation for the record

Event Timeline

MarkTraceur changed the title of this paste from untitled to For the record re: sleep 1 in browser tests.
MarkTraceur updated the paste's language from autodetect to autodetect.
MarkTraceur added a project: UploadWizard.