If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Reftests at layout/reftests/bugs/1133905* fail on Linux with APZ enabled

RESOLVED FIXED in Firefox 41

Status

()

Core
Panning and Zooming
RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: kats, Assigned: kats)

Tracking

unspecified
mozilla41
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox41 fixed)

Details

Attachments

(1 attachment)

These reftests are annotated as skip-if(!asyncPanZoom) so currently they are not running on desktop. Enabling APZ makes them run. However, the tests are actually assuming that the meta-viewport tags are being interpreted and the page zoom is being adjusted (see bug 1139306 comment 55 - 57). On desktop, enabling APZ doesn't automatically mean that the meta-viewport tag is applied, so I think the skip-if condition is wrong. We should just run these tests on B2G (i.e. on platforms where APZ and meta-viewport are both enabled).
Created attachment 8626440 [details] [diff] [review]
Patch
Attachment #8626440 - Flags: review?(tnikkel)
Comment on attachment 8626440 [details] [diff] [review]
Patch

If in the future we're going to have more cases of reftests needing metaviewport on/off we should maybe think about having a reftest environment variable for metaviewport support.
Attachment #8626440 - Flags: review?(tnikkel) → review+

Comment 3

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/f71d1e61111b
https://hg.mozilla.org/mozilla-central/rev/f71d1e61111b
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox41: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #0)
> We should just [only] run these
> tests on B2G (i.e. on platforms where APZ and meta-viewport are both
> enabled).

In bug 1307332, :Aryx is performing the monumental task of removing all B2G/Mulet reftest.list annotations (basically making our harness assume B2G/Mulet are never defined).

In part, that means all of the lines affected in this bug are removed -- which leaves all of the tests unused & worthy-of-being-deleted.

Can/should we delete them (and all the reftest.list lines), or should we enable them under some alternate constellation of annotations?
Flags: needinfo?(tnikkel)
(hoping kats [patch author] and/or tnikkel [patch-reviewer & test-author] will have some insight into comment 5.)
Flags: needinfo?(bugmail)
We should probably switch them from skip-if(!B2G) to skip-if(!Android). In theory the tests should run and pass fine on Fennec, although what happens in practice may be different.
Flags: needinfo?(bugmail)
What kats said.
Flags: needinfo?(tnikkel)
You need to log in before you can comment on or make changes to this bug.