Closed Bug 945765 Opened 6 years ago Closed 6 years ago
Add some basic apzc tests to catch regressions
No description provided.
Summary: Add some basic apzc test to catch regressions → Add some basic apzc tests to catch regressions
Status: NEW → ASSIGNED
Priority: -- → P2
QA Contact: jbecerra
Whiteboard: [beta28] p=0
Comment on attachment 8344795 [details] [diff] [review] two basic apzc tests v.1 Fixing this for good and adding some tests. (Note the tests rely on native events from bug 941774.)
Comment on attachment 8344795 [details] [diff] [review] two basic apzc tests v.1 argh! wrong bug/patch.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 29
Comment on attachment 8345930 [details] [diff] [review] basic apzc tests v.1 [Approval Request Comment] Bug caused by (feature/regressing bug #): no regression, adding some test coverage for apzc. User impact if declined: none. Testing completed (on m-c, etc.): yes. Risk to taking this patch (and alternatives if risky): none. String or IDL/UUID changes made by this patch: none.
Attachment #8345930 - Flags: approval-mozilla-aurora?
Attachment #8345930 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Could anyone give some guidelines for the QA to verify this? Thanks!
(In reply to Manuela Muntean [:Manuela] [QA] from comment #9) > Could anyone give some guidelines for the QA to verify this? Thanks! This bug is just adding automated tests, so there's no manual testing needed... I took a look at the m-c build logs on https://tbpl.mozilla.org/?tree=Mozilla-Aurora and https://tbpl.mozilla.org/ and verified that the new test file (browser_apzc_basic.js) is running in the Win8 "mc" test jobs.
You need to log in before you can comment on or make changes to this bug.