Temporarily disable APZ on beta

RESOLVED FIXED in Firefox 46

Status

()

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

People

(Reporter: Felipe, Assigned: Felipe)

Tracking

unspecified
mozilla47
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox46+ fixed, firefox47 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
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.
(Assignee)

Comment 1

2 years ago
Created attachment 8726833 [details] [diff] [review]
patch
Assignee: nobody → felipc
Status: NEW → ASSIGNED
Attachment #8726833 - Flags: review?(bugmail.mozilla)
(Assignee)

Comment 2

2 years ago
[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
status-firefox46: --- → affected
tracking-firefox46: --- → ?
(Assignee)

Updated

2 years ago
Blocks: 1253669
(Assignee)

Comment 3

2 years ago
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]: -
Attachment #8726833 - Flags: approval-mozilla-aurora?
Comment on attachment 8726833 [details] [diff] [review]
patch

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

r+ with appropriate commit message
Attachment #8726833 - Flags: review?(bugmail.mozilla) → review+

Comment 5

2 years ago
https://hg.mozilla.org/integration/fx-team/rev/92a3b28fa06a
Tracking for 46
tracking-firefox46: ? → +

Comment 7

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/92a3b28fa06a
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox47: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
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.
Attachment #8726833 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/releases/mozilla-aurora/rev/eb409b0a6253
status-firefox46: affected → fixed
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).
Flags: needinfo?(felipc)
(Assignee)

Comment 11

a year ago
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
Flags: needinfo?(felipc)
Blocks: 1257181
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.
You need to log in before you can comment on or make changes to this bug.