Closed
Bug 914890
Opened 12 years ago
Closed 12 years ago
While running a standalone mochitest, prefs and permissions should also reset after the mochitest run
Categories
(Testing :: Mochitest, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla27
People
(Reporter: martijn.martijn, Assigned: martijn.martijn)
Details
Attachments
(1 file, 1 obsolete file)
|
1.30 KB,
patch
|
Details | Diff | Splinter Review |
Currently, when a test file is using pushPermissions/pushPrefEnv, the prefs and permissions aren't removed after the mochitest is run, when it is being run standalone.
This can lead to failures when reloading the standalone mochitest.
The attached patch fixes that.
Attachment #802662 -
Flags: review?(jmaher)
|
||
Comment 1•12 years ago
|
||
(Yet another instance of bug 508664).
|
||
Comment 2•12 years ago
|
||
Comment on attachment 802662 [details] [diff] [review]
SimpleTest.js.diff
Review of attachment 802662 [details] [diff] [review]:
-----------------------------------------------------------------
I suspect there is other simple cleanup we could add as well.
Attachment #802662 -
Flags: review?(jmaher) → review+
|
Assignee | |
Comment 3•12 years ago
|
||
layout/base/tests/test_reftests_with_caret.html
layout/generic/test/test_bug496275.html
QA Contact: martijn.martijn
|
|
Assignee | |
Comment 4•12 years ago
|
||
(In reply to Joel Maher (:jmaher) from comment #2)
>
> I suspect there is other simple cleanup we could add as well.
Not sure what other cleanup you are thinking of.
Assignee: nobody → martijn.martijn
Attachment #802662 -
Attachment is obsolete: true
|
|
Assignee | |
Updated•12 years ago
|
Keywords: checkin-needed
QA Contact: martijn.martijn
|
||
Comment 5•12 years ago
|
||
Keywords: checkin-needed
|
|
||
Comment 6•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in
before you can comment on or make changes to this bug.
Description
•