945765 - Add some basic apzc tests to catch regressions

Status: VERIFIED FIXED

(Firefox for Metro Graveyard :: Tests, defect, P2)

Comment 1

[Jim Mathies [:jimm]]

Attachment: two basic apzc tests v.1

Comment 2

[Jim Mathies [:jimm]]

Attachment: two basic apzc tests v.1

Comment 3

[Jim Mathies [:jimm]]

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 4

[Jim Mathies [:jimm]]

Comment on attachment 8344795 [details] [diff] [review]
two basic apzc tests v.1

argh! wrong bug/patch.

Comment 5

[Jim Mathies [:jimm]]

Attachment: basic apzc tests v.1

Comment 6

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/4ad9d2b82a4c

Comment 7

[Jim Mathies [:jimm]]

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.

Comment 8

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/cb28a42ecacf

Comment 9

[Manuela Muntean [Away]]

Could anyone give some guidelines for the QA to verify this? Thanks!

Comment 10

[Matt Brubeck (:mbrubeck)]

(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.