246.84 KB, video/webm
4.05 KB, patch
|Details | Diff | Splinter Review|
There is a flicker drawing the new downloads panel and #social-notification-panel. Clicking on the download button or on one of the social provider's buttons should open a panel with a slight (20px) animation of a translateY but this recently started looking quite bad with a lot of flickering. Note that occasionally the flicker is not noticeable. See the attached screen recording of expected and actual behaviours but note that the flickering doesn't seem as bad in the video. Last good nightly: 2013-01-23 First bad nightly: 2013-01-24 Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=2876e73c9b6f&tochange=98ea4d294369 I suspected the cause was bug 832641 (http://hg.mozilla.org/mozilla-central/rev/2d5b54f00e2e ) so I backed that patch out locally and the problem went away. That patch was uplifted all the way to beta (19). changeset: 119576:2d5b54f00e2e user: Matt Woodrow <firstname.lastname@example.org> date: Wed Jan 23 18:46:36 2013 +1300 summary: Bug 832641 - Invalidate popups when they become visible rather than handling invalidations while they are hidden. r=roc
Thanks - given where we are in the cycle and the severity of this issue, this is a wontfix for FF19. Would like to hear Gavin's opinion on tracking for FF20 and up. I think we'd take a low risk fix, but this likely doesn't need to track there either.
We may be able to workaround this for affected panels by removing the transition but I haven't confirmed this.
I think we should track for investigation in 20.
Assignee: nobody → matt.woodrow
We were correctly invalidating and scheduling a paint, but we were compositing the old content before the refresh driver ran and updated it. This seems to work, but I don't know anything about the cocoa widget code.
Attachment #715323 - Flags: review?(roc)
Attachment #715323 - Flags: review?(roc) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
Looks like we need an uplift nomination here - also asking QA to verify the fix on nightly.
Comment on attachment 715323 [details] [diff] [review] Make WillPaintWindow work for cocoa popups [Approval Request Comment] Bug caused by (feature/regressing bug #): Bug 832641 User impact if declined: Flicking popups with the social sidebar Testing completed (on m-c, etc.): Been on m-c for a week Risk to taking this patch (and alternatives if risky): Reasonably low risk, no alternative fix known. String or UUID changes made by this patch: None
Comment on attachment 715323 [details] [diff] [review] Make WillPaintWindow work for cocoa popups Already approving the low risk patch on aurora to get more test coverage in preparation for beta uplift(before 3/12) once QA verifies on nightly/aurora.
Attachment #715323 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(In reply to Lukas Blakk [:lsblakk] from comment #7) > asking QA to verify the fix on nightly. I confirm the fix is verified on Latest Nightly (2013-03-07) Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:22.0) Gecko/20130307 Firefox/22.0 (20130307030926)
(In reply to Ryan VanderMeulen [:RyanVM] from comment #11) > https://hg.mozilla.org/releases/mozilla-aurora/rev/9b6f8360af61 Ryan is in plan a push for beta too?
Attachment #715323 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
(In reply to MarioMi (:MarioMi) from comment #12) > (In reply to Ryan VanderMeulen [:RyanVM] from comment #11) > > https://hg.mozilla.org/releases/mozilla-aurora/rev/9b6f8360af61 > > Ryan is in plan a push for beta too? This is going to require more fixing up for beta than I'm comfortable doing. Matt's going to have to handle the uplift.
I confirm the fix is verified on FF 20b5 and Latest Aurora too. Mac OS 10.7: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:20.0) Gecko/20100101 Firefox/20.0(20130313170052) Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:21.0) Gecko/20130313 Firefox/21.0(20130313042013)
Based on Comment 10 and Comment 15:
Sorry for last reply it automatically changed Status while trying to clear "verifyme" from keywords.
You need to log in before you can comment on or make changes to this bug.