Closed Bug 890720 Opened 11 years ago Closed 11 years ago

Remove unused user shutdown timeouts in restart tests

Categories

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Product:
Mozilla QA Graveyard
Component:
Mozmill Tests
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

(firefox24 wontfix, firefox25 fixed)

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox24 --- wontfix
firefox25 --- fixed

People

(Reporter: jfrench, Assigned: jfrench)

References

(
URL
)

Details

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

Attachments

(1 file)

remove unused shutdown timeouts (default) rev1.0
1.92 KB, patch
whimboo
: review+
Details | Diff | Splinter Review 
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.
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)
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
Closed: 11 years ago
status-firefox24: --- → wontfix
status-firefox25: --- → fixed
Resolution: --- → FIXED
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: