Temporarily disable APZ on beta

RESOLVED FIXED in Firefox 46

Status

()

RESOLVED FIXED
3 years ago
3 years 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

3 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

3 years ago
Posted patch patchSplinter Review
Assignee: nobody → felipc
Status: NEW → ASSIGNED
Attachment #8726833 - Flags: review?(bugmail.mozilla)
(Assignee)

Comment 2

3 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

3 years ago
Blocks: 1253669
(Assignee)

Comment 3

3 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+
Tracking for 46
tracking-firefox46: ? → +

Comment 7

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/92a3b28fa06a
Status: ASSIGNED → RESOLVED
Last Resolved: 3 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+
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

3 years 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)
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.