https://treeherder.mozilla.org/#/jobs?repo=try&revision=7ecce716a4ea50bbb6b7219fe8cf12c202795a78&group_state=expanded&filter-resultStatus=success&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception&filter-resultStatus=retry&filter-resultStatus=running&filter-resultStatus=pending&filter-resultStatus=runnable&selectedJob=156136564 > TEST-UNEXPECTED-FAIL | devtools/server/tests/browser/browser_canvasframe_helper_04.js | Test timed out -
actually, browser_canvasframe_helper_04.js is disabled on e10s
looks like, the content is not yet ready for synthesizeMouseDown after load event? the remaining part after `await loaded` is executed in the following microtask checkpoint. https://hg.mozilla.org/try/rev/4133d526893dd9f9e450f30866d49690db5facc3#l10.18 but I think it's essentially not changed by the patch.
without the bug 1193394 patch, the callstack for the remaining part after `await loaded` is different. it's outside nsDocumentViewer::LoadComplete.
So, it just need to wait for one event tick, to avoid running the remaining part in the microtask checkpoint for the load event itself, to make sure the mouse event is properly synthesized.
Assignee: nobody → arai.unmht
Status: NEW → ASSIGNED
Just like bug 1430408, added wait after load event. here I used setTimeout since it's inside content process and it doesn't have executeSoon
Attachment #8943924 - Flags: review?(pbrosset)
Comment on attachment 8943924 [details] [diff] [review] Wait for the next event tick after load event in devtools/server/tests/browser/browser_canvasframe_helper_04.js Review of attachment 8943924 [details] [diff] [review]: ----------------------------------------------------------------- Seems good to me. Thank you.
Attachment #8943924 - Flags: review?(pbrosset) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/f2df9f222906 Wait for the next event tick after load event in devtools/server/tests/browser/browser_canvasframe_helper_04.js r=pbro
https://hg.mozilla.org/integration/mozilla-inbound/rev/f2df9f22290658f15068ebdfd3f1ecee377e88f7 Bug 1430383 - Wait for the next event tick after load event in devtools/server/tests/browser/browser_canvasframe_helper_04.js r=pbro
You need to log in before you can comment on or make changes to this bug.