Closed Bug 999604 Opened 10 years ago Closed 10 years ago

fix browser-chrome tests in toolkit/content/ to not replace the original tab

Categories

(Toolkit :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX
Tracking Status
firefox30 --- fixed
firefox31 --- fixed
firefox32 --- wontfix

People

(Reporter: froydnj, Assigned: froydnj)

Details

Attachments

(1 file)

This causes problems for tests coming after these, let's fix this.
Mostly just adding a tab before executing everything, so we can remove the
newly added tab at the end, instead of the pre-existing tab.
Attachment #8410452 - Flags: review?(ehsan)
Blocks: 805068
Comment on attachment 8410452 [details] [diff] [review]
fix browser-chrome tests in toolkit/content/ to not replace the original tab

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

I guess this is ok.
Attachment #8410452 - Flags: review?(ehsan) → review+
Flags: needinfo?(ryanvm)
Assignee: nobody → nfroyd
https://hg.mozilla.org/mozilla-central/rev/a09de35d4423
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
Given bug 805068, I reverted this and removed the redundant cleanup code:
https://hg.mozilla.org/integration/fx-team/rev/2af97018bc81
No longer blocks: 805068
Merge of backout:
https://hg.mozilla.org/mozilla-central/rev/2af97018bc81
Resolution: FIXED → WONTFIX
Target Milestone: mozilla31 → ---
You need to log in before you can comment on or make changes to this bug.