1253667 - Temporarily disable APZ on beta

Status: RESOLVED FIXED

(Core :: Panning and Zooming, defect)

Description

[:Felipe Gomes (needinfo for replies!)]

It was decided that, at least for the first part of the e10s experiment on beta46, we want to do it with APZ turned off.  It's on right now (bug 1178298), and it's better to have it turned off in the tree rather than let the experiment manage it.

Comment 1

[:Felipe Gomes (needinfo for replies!)]

Attachment: patch

Comment 2

[:Felipe Gomes (needinfo for replies!)]

[Tracking Requested - why for this release]:
It was agreed on the e10s meeting today that apz should be turned off during the initial tests of e10s for beta46

Comment 3

[:Felipe Gomes (needinfo for replies!)]

Comment on attachment 8726833 [details] [diff] [review]
patch

Approval Request Comment
[Feature/regressing bug #]: disable apz for e10s testing on 46
[User impact if declined]: apz will be on for e10s users
[Describe test coverage new/current, TreeHerder]: -
[Risks and why]: patch just switches the pref
[String/UUID change made/needed]: -

Comment 4

[Kartikaya Gupta (email:[e-mail])]

Comment on attachment 8726833 [details] [diff] [review]
patch

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

r+ with appropriate commit message

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/92a3b28fa06a

Comment 6

[Liz Henry (:lizzard) (relman/hg->git project)]

Tracking for 46

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/92a3b28fa06a

Comment 8

[Liz Henry (:lizzard) (relman/hg->git project)]

Comment on attachment 8726833 [details] [diff] [review]
patch

From discussion on Friday, we plan to disable apz on beta 46 for at least a couple of weeks. Please uplift to aurora.

Comment 9

[Kartikaya Gupta (email:[e-mail])]

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

Comment 10

[Kartikaya Gupta (email:[e-mail])]

Felipe, we should back this out on Beta soonish, right? When does it need to land on beta to miss beta 2 but make it in beta 3 (assuming those are the relevant betas).

Comment 11

[:Felipe Gomes (needinfo for replies!)]

Kats, I just talked with Erin, and the plan is to make it to beta 4, which goes to build on the 21st. So if it's agreed to turn APZ for the next phase of the experiment, we should back out during the weekend

Comment 12

[Kartikaya Gupta (email:[e-mail])]

Thanks. I filed bug 1257181 to reenable, since I felt it would be cleaner to do it as a new bug rather than a backout in this bug.