Closed
Bug 1253667
Opened 9 years ago
Closed 9 years ago
Temporarily disable APZ on beta
Categories
(Core :: Panning and Zooming, defect)
Core
Panning and Zooming
Tracking
()
RESOLVED
FIXED
mozilla47
People
(Reporter: Felipe, Assigned: Felipe)
References
Details
Attachments
(1 file)
(deleted),
patch
|
kats
:
review+
lizzard
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
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•9 years ago
|
||
Assignee: nobody → felipc
Status: NEW → ASSIGNED
Attachment #8726833 -
Flags: review?(bugmail.mozilla)
Assignee | ||
Comment 2•9 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 | ||
Comment 3•9 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 4•9 years ago
|
||
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
Comment 7•9 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 9 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+
Comment 9•9 years ago
|
||
Comment 10•9 years ago
|
||
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•9 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)
Comment 12•9 years ago
|
||
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.
Description
•