Closed Bug 1235653 Opened 8 years ago Closed 8 years ago

Enable APZ on Mulet

Categories

(Firefox OS Graveyard :: Runtime, defect)

Product:
Firefox OS Graveyard
Component:
Runtime
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: kats, Unassigned)

References

Details

APZ is enabled pretty much everywhere except on Mulet. We should enable it on Mulet. However, enabling it causes a ton of reftest failures (most of which look scrollbar-related, and are probably trivial to fix) and a bunch of Gij failures. Until bug 1228636 is fixed, it's not really worth spending time on this since it's going to be painful to get the tests passing. Filing this bug anyway as a placeholder.
OS: Gonk (Firefox OS) → Linux
Hardware: ARM → x86
With bug 1228636 fixed I can run reftests on Mulet locally. However I can't reproduce the reftest failures in gfx/layers/apz/test/reftest/ - I get 6 passes and 3 skips.
Also for the record the last time I did a try push with APZ enabled on Mulet was at https://treeherder.mozilla.org/#/jobs?repo=try&revision=0e7d8a488961
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #1)
> With bug 1228636 fixed I can run reftests on Mulet locally. However I can't
> reproduce the reftest failures in gfx/layers/apz/test/reftest/ - I get 6
> passes and 3 skips.

This was when I was running the mulet reftests with --e10s. Without --e10s I can repro the failures. However the APZ check for e10s at [1] is inside a !defined(MOZ_B2G) so that's not the problem; it must be somewhere else.

[1] https://dxr.mozilla.org/mozilla-central/rev/6020a4cb41a77a09484c24a5875bb221714c0e6a/gfx/thebes/gfxPlatform.cpp#2178
Is Mulet still a thing or should this be WONTFIX?
I don't know if it's still a thing. I assume not, but I'd rather "fix" it than "wontfix" it, i.e. remove the ifdef guard at http://mxr.mozilla.org/mozilla-central/source/browser/app/profile/firefox.js?rev=b2bc8ace9ca2#1434
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.