Closed
Bug 674315
Opened 13 years ago
Closed 13 years ago
move test_clear_site_data.html to mochitest-chrome
Categories
(Core Graveyard :: Plug-ins, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla8
People
(Reporter: jmaher, Assigned: jmaher)
Details
(Whiteboard: [specialpowers][inbound])
Attachments
(1 file)
3.21 KB,
patch
|
Dolske
:
review+
|
Details | Diff | Splinter Review |
in working on porting PluginUtils.js to use SpecialPowers, I found that the 2 tests which used it did a lot of interaction which required enabledPrivileges. I would like to move these from mochitest-plain to mochitest-chrome.
Assignee | ||
Comment 1•13 years ago
|
||
Assignee | ||
Comment 2•13 years ago
|
||
can we verify that this test will still gain the same coverage as a mochitest-chrome and that the patch looks good.
Updated•13 years ago
|
Attachment #548547 -
Flags: review?(benjamin) → review?(dolske)
Comment 3•13 years ago
|
||
Comment on attachment 548547 [details] [diff] [review] move test_clear_site_data.html to mochitest-chrome (1.0) In general, I'd be wary of running content stuff in chrome (since that basically never happens in normal usage), but that shouldn't impact anything here. The content part of this test is basically incidental to what it's doing, anyway.
Attachment #548547 -
Flags: review?(dolske) → review+
Assignee | ||
Updated•13 years ago
|
Whiteboard: [specialpowers] → [specialpowers][inbound]
Comment 4•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/02d8eaab6cd3
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
Updated•2 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•