1421117 - Wait for the next event tick before resolving promise in browser_oneOffHeader.js.

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Tooru Fujisawa [:arai]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=b5800bb624450f22893e4918d653b3dbe231c4ac&selectedJob=147752924
> TEST-UNEXPECTED-FAIL | browser/components/search/test/browser_oneOffHeader.js | Header has the correct index selected when a search engine has been selected - Got 0, expected 2

after fixing bug 1193394, the test needs to wait for the next event tick before resolving promise in mousemove event, to make sure everything gets ready.

Comment 1

[Tooru Fujisawa [:arai]]

Attachment: Wait for the next event tick before resolving promise in browser_oneOffHeader.js.

synthesizeNativeMouseMove should wait for the next event tick before resolving the promise inside mousemove event handler,
to avoid the remaining part of the testcase runs unexpectedly earlier,
after the change in bug 1193394.

Comment 2

[:Gijs (out for now; he/him)]

Comment on attachment 8932840 [details] [diff] [review]
Wait for the next event tick before resolving promise in browser_oneOffHeader.js.

Review of attachment 8932840 [details] [diff] [review]:
-----------------------------------------------------------------

Florian would be a better reviewer here.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5513671525ed
Wait for the next event tick before resolving promise in browser_oneOffHeader.js. r=florian

Comment 4

[Tooru Fujisawa [:arai]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/5513671525edd09ee5f1235087fd6ed7ca800cbe
Bug 1421117 - Wait for the next event tick before resolving promise in browser_oneOffHeader.js. r=florian

Comment 5

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/5513671525ed