Closed Bug 887543 Opened 11 years ago Closed 11 years ago

make profiledir in head_docshell.js use do_get_profile()

Categories

(Core :: DOM: Navigation, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla25

People

(Reporter: mihneadb, Assigned: mihneadb)

References

Details

Attachments

(2 files, 3 obsolete files)

Needing this to run the docshell xpcshell tests in parallel.
Assignee: nobody → mihneadb
Blocks: 887064
Comment on attachment 768051 [details] [diff] [review]
use do_get_profile in head_docshell.js

You can go a little further here. do_get_profile does all that "register provider" stuff, so you can remove that as well:
http://mxr.mozilla.org/mozilla-central/source/testing/xpcshell/head.js#917

You also don't need the profileDir.append line, since there's no need to make a subdir.
Attachment #768051 - Attachment is obsolete: true
Attachment #772424 - Flags: review?(ted)
Attachment #772424 - Flags: review?(ted) → review+
Attached patch use do_get_profile (obsolete) — Splinter Review
Comment on attachment 772772 [details] [diff] [review]
use do_get_profile

added commit metadata
Attachment #772772 - Attachment description: make profiledir in head_docshell.js use do_get_profile(), → use do_get_profile
Attachment #772772 - Flags: review+
Attachment #772424 - Attachment is obsolete: true
Keywords: checkin-needed
Attachment #772772 - Attachment is obsolete: true
That fixed the Windows debug problems
https://hg.mozilla.org/mozilla-central/rev/392293e24550
https://hg.mozilla.org/mozilla-central/rev/c8f6d17961cf
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: