Closed Bug 538875 Opened 15 years ago Closed 15 years ago

Remove |mainController|, use mc everywhere instead

Categories

(Thunderbird :: Testing Infrastructure, defect)

defect
Not set
minor

Tracking

(thunderbird3.0 .2-fixed)

RESOLVED FIXED
Thunderbird 3.1a1
Tracking Status
thunderbird3.0 --- .2-fixed

People

(Reporter: rain1, Assigned: rain1)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
We never use mainController anywhere in folder-display, and keeping it around is just a potential source for errors.

I've gone ahead and changed the other tests to use |mc| as well.
Attachment #420967 - Flags: review?(bugmail)
Version: 3.0 → Trunk
Attachment #420967 - Flags: review?(bugmail) → review+
http://hg.mozilla.org/comm-central/rev/3b0bb34f73a6
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.1a1
Comment on attachment 420967 [details] [diff] [review]
patch

npotb, simple renaming. This will also make the patch in bug 538855 directly applicable if we decide to land that on comm-1.9.1.
Attachment #420967 - Flags: approval-thunderbird3.0.2?
Attachment #420967 - Flags: approval-thunderbird3.0.2? → approval-thunderbird3.0.2+
Verifying as fixed for 3.0.2 based on the fact that these are test-only patches and the last 5 runs from MozMill/unit tests have been green with no failures relating to these patches.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: