Closed Bug 1381097 Opened 2 years ago Closed 10 months ago
Popups with APZ enabled to not repaint correctly after resize
59 bytes, text/x-review-board-request
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:56.0) Gecko/20100101 Firefox/56.0 Build ID: 20170714030205 Steps to reproduce: 1. Download the Feedly Notifier add-on (https://olsh.me/Feedly-Notifier/). 2. Click on the icon that the add-on puts in the navigation toolbar. Actual results: The panel appears for a few milliseconds starting to animate out from the button, but then it suddenly disappears. Expected results: The panel should have opened, showing either a Login button or a list of all of your Feedly reading list. I have noticed this bug since the 2017-07-11 build of Nightly.
Component: Untriaged → WebExtensions: Frontend
Product: Firefox → Toolkit
Component: WebExtensions: Frontend → Graphics: Layers
Product: Toolkit → Core
Apparently this is due to APZ being enabled for the popups, regardless of whether they have remote content.
Summary: OOP extensions popups do not repaint correctly after resize → Popups with APZ enabled to not repaint correctly after resize
Comment on attachment 8886804 [details] Bug 1381097: Part 1 - Add default-false pref to enable APZ in remote popups. https://reviewboard.mozilla.org/r/157582/#review162884
Attachment #8886804 - Flags: review?(bugmail) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/228602d59911f40a39c75841722be7c4b124c447 Bug 1381097: Part 1 - Add default-false pref to enable APZ in remote popups. r=kats
The problem is gone after this patch on my machine, Kris should we close this bug? or there are some reason that we should leave it opened?
2 years ago
When you get to this - let's decide if this can be closed or not.
The short term "fix" was to disable APZ in these popups. In the long term, we really want OOP popups to use APZ, so this will still need to be fixed in order to enable that.
(In reply to Kris Maglione [:kmag] (long backlog; ping on IRC if you're blocked) from comment #9) > The short term "fix" was to disable APZ in these popups. In the long term, > we really want OOP popups to use APZ, so this will still need to be fixed in > order to enable that. Hmmm. For me having apz.allow_zooming enabled is what broke the webext button menus (bug 1416207). In contrast I have apz.popups.enabled enabled right now and the menus work.
That's not surprising. The problems are sporadic.
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → WONTFIX
Resolution: WONTFIX → FIXED
10 months ago
Assignee: nobody → kmaglione+bmo
10 months ago
You need to log in before you can comment on or make changes to this bug.