Closed
Bug 1452479
Opened 6 years ago
Closed 6 years ago
Mention script timeout duration in ScriptTimeoutError message
Categories
(Remote Protocol :: Marionette, enhancement, P3)
Remote Protocol
Marionette
Tracking
(firefox61 fixed)
RESOLVED
FIXED
mozilla61
| Tracking | Status | |
|---|---|---|
| firefox61 | --- | fixed |
People
(Reporter: ato, Assigned: ato)
Details
Attachments
(1 file)
We don’t currently log what the script timeout was when throwing ScriptTimeoutError. It would provide more value to the user to include the timeout duration than to record "Timed out" as we do currently.
|
Assignee | |
Updated•6 years ago
|
Assignee: nobody → ato
Status: NEW → ASSIGNED
| Comment hidden (mozreview-request) |
|
|
Assignee | |
Updated•6 years ago
|
Priority: -- → P3
|
||
Comment 2•6 years ago
|
||
| mozreview-review | ||
Comment on attachment 8966042 [details] Bug 1452479 - Include timeout duration in ScriptTimeoutError. https://reviewboard.mozilla.org/r/234764/#review240432
Attachment #8966042 -
Flags: review+
|
|
Assignee | |
Updated•6 years ago
|
Attachment #8966042 -
Flags: review?(mjzffr)
Pushed by atolfsen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/32f7ea5fb4ba Include timeout duration in ScriptTimeoutError. r=whimboo
|
||
Comment 4•6 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/32f7ea5fb4ba
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
|
||
Updated•1 year ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•