Fix script_timeout regressions for Execute (Async) Script since the landing of bug 1510929
Categories
(Remote Protocol :: Marionette, defect, P1)
Tracking
(firefox-esr60 unaffected, firefox65 unaffected, firefox66 fixed, firefox67 fixed)
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox65 | --- | unaffected |
firefox66 | --- | fixed |
firefox67 | --- | fixed |
People
(Reporter: whimboo, Assigned: whimboo)
References
Details
(Keywords: regression)
Attachments
(2 files)
Since bug 1510929 landed both the execute script and execute async script are broken in the following ways:
-
The custom timeout as set isn't reset if the script runs into a timeout
-
Fractions of a second are lost by
script_timeout / 1000
because it's a decimal division. As such a timeout of 100ms will result in 0ms.
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Comment 2•6 years ago
|
||
This fixes the following regressions as introduced by
bug 1510929 for the Marionette client.
-
The custom timeout as set isn't reset if the
script times out. -
Fractions of a second for the script timeout are
lost because the division operates on decimal valus.
As such a timeout of 100ms will result in 0ms.
Assignee | ||
Comment 3•6 years ago
|
||
Fractions of a second are lost because the division
in getting the timeout value operates on decimal valus.
As such a timeout of 100ms will result in 0ms.
Depends on D18214
Comment 5•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/934099e1c866
https://hg.mozilla.org/mozilla-central/rev/71f896682a2f
Assignee | ||
Comment 6•6 years ago
|
||
Please uplift the test-only patch to mozilla-beta to fix this regression.
Comment 7•6 years ago
|
||
:whimboo can you, please, point me to the test-only patch?
Comment 8•6 years ago
|
||
bugherder uplift |
Assignee | ||
Updated•6 years ago
|
Updated•6 years ago
|
Updated•6 years ago
|
Updated•2 years ago
|
Description
•