Closed Bug 883634 Opened 6 years ago Closed 6 years ago

harness-overlay.xul is unnecessary since bug 654461

Categories

(Testing :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla24

People

(Reporter: Gavin, Assigned: Gavin)

Details

Attachments

(1 file)

It's only used to overlay testing/mochitest/harness.xul, AFAICT, so it might as well just be included there rather than in a separate overlay.
Attached patch patchSplinter Review
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Attachment #763252 - Flags: review?(jmaher)
Comment on attachment 763252 [details] [diff] [review]
patch

Review of attachment 763252 [details] [diff] [review]:
-----------------------------------------------------------------

it appears as if you are just moving harness-overlay.xul -> harness.xul.  I prefer fewer files (i.e. this patch), but is there any other changes to this patch or in the works to depend on this?
Attachment #763252 - Flags: review?(jmaher) → review+
No, it's just cleanup.
great, less files == less confusion!
Flags: in-testsuite-
https://hg.mozilla.org/mozilla-central/rev/a57608d6e455
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.