Closed Bug 608507 Opened 14 years ago Closed 14 years ago

some minor test cleanup

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set
trivial

Tracking

()

VERIFIED FIXED
mozilla2.0b7

People

(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)

Details

Attachments

(1 file)

Just a couple of more undeclared variables and an indentation fix I found.
Attached patch patchSplinter Review
Attachment #487148 - Flags: review?(dtownsend)
Severity: normal → trivial
Status: NEW → ASSIGNED
Attachment #487148 - Flags: review?(dtownsend) → review+
Pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/2dba8f58b228
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite-
Flags: in-litmus-
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b7
Robert, why some files are using 'let' while others got 'var'?
Before the patch some of the files were using var vs. let so to be consistent the patch uses var when the pre-existing test used var and used let when the pre-existing test used let. This made the patch consistent with the pre-existing code in the test.
That makes sense. Thanks for the explanation. Marking as verified fixed by check-in.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: