Remove unused user shutdown timeouts in restart tests

RESOLVED FIXED

Status

Mozilla QA
Mozmill Tests
--
minor
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jfrench, Assigned: jfrench)

Tracking

unspecified

Firefox Tracking Flags

(firefox24 wontfix, firefox25 fixed)

Details

(Whiteboard: [mentor=whimboo][lang=js][good first bug], URL)

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
I noticed a couple of unused timeouts in restart tests. TIMEOUT_USERSHUTDOWN, and TIMEOUT_USER_SHUTDOWN, respectively in:

functional/restartTests/testAddons_enableDisableExtension/test4.js
functional/restartTests/testAddons_uninstallExtension/test3.js

This bug will represent their removal.
(Assignee)

Comment 1

4 years ago
Created attachment 771859 [details] [diff] [review]
remove unused shutdown timeouts (default) rev1.0

Patch "remove unused timeouts (default) rev1.0" for the Default branch. Two files modified.

Tested with Default Nightly 25.0a1 20130707031138 with both mozmill 1.5.21 and 2.0rc4 latest. Tests pass as expected.
Attachment #771859 - Flags: review?(hskupin)
(Assignee)

Updated

4 years ago
Severity: normal → minor
Attachment #771859 - Flags: review?(hskupin) → review+
Thanks Jonathan! Landed as:

http://hg.mozilla.org/qa/mozmill-tests/rev/ed982dd9b8a9
Status: NEW → RESOLVED
Last Resolved: 4 years ago
status-firefox24: --- → wontfix
status-firefox25: --- → fixed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.