Closed
Bug 1257181
Opened 9 years ago
Closed 9 years ago
Re-enable APZ on beta
Categories
(Core :: Panning and Zooming, defect)
Tracking
()
RESOLVED
FIXED
mozilla48
People
(Reporter: kats, Assigned: kats)
References
Details
(Keywords: feature, Whiteboard: [gfx-noted])
Attachments
(1 file)
(deleted),
patch
|
Felipe
:
review+
lizzard
:
approval-mozilla-aurora+
lizzard
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #1253667 +++
It was decided that, after doing the first few 46 beta builds with APZ off, we would turn it back on in beta4 for a couple of betas, and then the e10s experiment in 46 beta would end.
So basically this bug is to back out bug 1253667 and land it on beta for 46.0b4. See bug 1253667 comment 11 as well.
Assignee | ||
Updated•9 years ago
|
status-firefox46:
--- → affected
status-firefox47:
--- → affected
Assignee | ||
Comment 1•9 years ago
|
||
Approval Request Comment
[Feature/regressing bug #]: APZ
[User impact if declined]: APZ won't be enabled in beta for the e10s+APZ experiment
[Describe test coverage new/current, TreeHerder]: all the APZ stuff
[Risks and why]: This was discussed in the e10s cross-functional
[String/UUID change made/needed]: none
Attachment #8731250 -
Flags: review?(felipc)
Attachment #8731250 -
Flags: approval-mozilla-beta?
Attachment #8731250 -
Flags: approval-mozilla-aurora?
Comment 2•9 years ago
|
||
kats, just to be clear, we plan to disable e10s on Beta 46 after the e10s+APZ experiment ends, which will implicitly disable APZ (regardless of the value of the APZ pref). Is that still the plan?
status-firefox45:
--- → wontfix
Flags: needinfo?(bugmail.mozilla)
Comment on attachment 8731250 [details] [diff] [review]
Patch
OK to turn on apz by default for aurora and beta.
This should make it into beta 4.
Attachment #8731250 -
Flags: approval-mozilla-beta?
Attachment #8731250 -
Flags: approval-mozilla-beta+
Attachment #8731250 -
Flags: approval-mozilla-aurora?
Attachment #8731250 -
Flags: approval-mozilla-aurora+
Comment 5•9 years ago
|
||
Comment on attachment 8731250 [details] [diff] [review]
Patch
Review of attachment 8731250 [details] [diff] [review]:
-----------------------------------------------------------------
please hold off landing on beta until we get agreement at the meeting tomorrow, and also because I want to make sure this only gets into the right beta and not earlier. If you want to land on aurora sooner, go ahead.
Attachment #8731250 -
Flags: review?(felipc) → review+
Assignee | ||
Comment 6•9 years ago
|
||
I'll land this on inbound/aurora/beta once bug 1247854 and it's dependencies is safely on beta. I'll be around this weekend and can do it then if needed.
Assignee | ||
Comment 8•9 years ago
|
||
I verified bug 1247854 is fixed on the build that has the uplift, so I landed the APZ-enabling patch:
https://hg.mozilla.org/releases/mozilla-aurora/rev/6d030cb1d2ab
https://hg.mozilla.org/releases/mozilla-beta/rev/02bdd0469632
Looks like the tag for FIREFOX_46_0b2_BUILD3, FIREFOX_46_0b2_RELEASE already happened so this will be in beta 3 as discussed during the meeting. I hope that's the right beta, not sure if plans are changing because of the updates being turned off.
Comment 9•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Comment 10•9 years ago
|
||
oh no, apz and the regressions:
https://treeherder.mozilla.org/perf.html#/alerts?id=566
and a compare view:
https://treeherder.mozilla.org/perf.html#/compare?originalProject=mozilla-beta&originalRevision=24b6aaac8973&newProject=mozilla-beta&newRevision=02bdd0469632&framework=1
these are all e10s regressions, that is even scarier!
Comment 11•9 years ago
|
||
:kats, i assume this was expected? please confirm, or outline the next steps.
Flags: needinfo?(bugmail.mozilla)
Assignee | ||
Comment 12•9 years ago
|
||
Yeah, this is expected. APZ is not riding to release on 46; we are tracking the regressions on Nightly and fixing them there.
Flags: needinfo?(bugmail.mozilla)
You need to log in
before you can comment on or make changes to this bug.
Description
•