1414170 - Fix timeout of toolkit/components/extensions/test/mochitest/test_ext_new_tab_processType.html that replies non-comformant Promise handling

Status: RESOLVED FIXED

(WebExtensions :: General, defect, P5)

Description

[Bevis Tseng[:bevis][:btseng](Exited)]

This is a follow-up of the try result in bug 1193394 comment 48:
(M15 in linux64-debug-build: https://treeherder.mozilla.org/logviewer.html#?job_id=141558476&repo=try&lineNumber=13913)
[test_ext_new_tab_processType.html]
TEST-UNEXPECTED-FAIL | toolkit/components/extensions/test/mochitest/test_ext_new_tab_processType.html | Test timed out.
TEST-UNEXPECTED-FAIL | toolkit/components/extensions/test/mochitest/test_ext_new_tab_processType.html | Extension left running at test shutdown
/tests/toolkit/components/extensions/test/mochitest/test_ext_new_tab_processType.html logged result after SimpleTest.finish(): Extension left running at test shutdown
TEST-UNEXPECTED-FAIL | toolkit/components/extensions/test/mochitest/test-oop-extensions/test_ext_new_tab_processType.html | Test timed out.
TEST-UNEXPECTED-FAIL | toolkit/components/extensions/test/mochitest/test-oop-extensions/test_ext_new_tab_processType.html | Extension left running at test shutdown
/tests/toolkit/components/extensions/test/mochitest/test-oop-extensions/test_ext_new_tab_processType.html logged result after SimpleTest.finish(): Extension left running at test shutdown

Comment 1

[Bevis Tseng[:bevis][:btseng](Exited)]

Take this bug to follow-up.

Comment 2

[Bevis Tseng[:bevis][:btseng](Exited)]

The root cause is the same as bug 1420059 comment 2.

Comment 3

[Bevis Tseng[:bevis][:btseng](Exited)]

Attachment: (v1) Patch: Fix timeout of toolkit/components/extensions/test/mochitest/test_ext_new_tab_processType.html that replies non-comformant Promise handling.

The root cause is the same as explained in bug 1420059 comment 2 that we should trigger synthesizeMouseAtCenter() until next tick of the eventloop.

Try result looks good with/without the microtask fix:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=bb591de8a50e212b0ccbdf4a1782a5af047fb3a3
https://treeherder.mozilla.org/#/jobs?repo=try&revision=df51aea8b00934db94e3699dae61369da9013448

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1dfa2eb559ad
Fix timeout of toolkit/components/extensions/test/mochitest/test_ext_new_tab_processType.html that replies non-comformant Promise handling. r=smaug

Comment 5

[Eliza Balazs [:ebalazs_]]

https://hg.mozilla.org/mozilla-central/rev/1dfa2eb559ad

Comment 6

[Eliza Balazs [:ebalazs_]]

https://hg.mozilla.org/mozilla-central/rev/1dfa2eb559ad