Closed Bug 978045 Opened 6 years ago Closed 6 years ago

Move plugin tests in browser/base/content/tests/general to browser/base/content/tests/plugins

Categories

(Firefox :: General, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
Firefox 30

People

(Reporter: gfritzsche, Assigned: gfritzsche)

References

Details

Attachments

(1 file, 1 obsolete file)

I often find myself wanting to run just the plugin/click-to-play tests to verify my changes - running all the general/ tests still takes too long.

I would like to move all plugin related tests (mostly browser_CTP_*, browser_pluginnotification.js, ...) to:
browser/base/content/test/plugins
jaws, any objections?
Flags: needinfo?(jaws)
Version: unspecified → Trunk
Status: NEW → ASSIGNED
go for it!
Flags: needinfo?(jaws)
Comment on attachment 8384566 [details] [diff] [review]
move bc plugin tests to browser/base/content/test/plugins/

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

rs=me, assuming you make sure that these are running on tbpl and all of the tests pass.
Attachment #8384566 - Flags: review+
The social/ tests have a dependency on general/blockNoPlugins.xml, which moved.
Attachment #8384566 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/9d369fc64e91
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 30
QA Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.