Closed Bug 631662 Opened 12 years ago Closed 12 years ago
We are painting thumbnails while in the background
We receive the events when the user is not in the Panorama UI, but we're supposed to just queue them up for later handling; it's not till you reenter the Panorama UI that thumbnails are supposed to be actually updated. bug 625561 has evidence that we're actually updating thumbnails while not in the Panorama UI. Looking at the code, we don't have an explicit check for whether you're in TabView; we just rely on paintingPaused, which should be set to true when TabView is hidden, but maybe it's not? This needs to be fixed due to its perf impact on regular pages.
Confirming. We're even painting twice when switching tabs.
Assignee: nobody → tim.taubert
Status: NEW → ASSIGNED
Version: unspecified → Trunk
Summary: Are we painting thumbnails while in the background? → We are painting thumbnails while in the background
Hardblocker - panorama should not impact perf for people who never use it. Can we get a patch up here right away?
blocking2.0: ? → final+
Yep, it's almost done.
(In reply to comment #2) > Hardblocker - panorama should not impact perf for people who never use it. Can > we get a patch up here right away? Oh and to be clear: this is not affecting users that never opened panorama.
Comment on attachment 509933 [details] [diff] [review] patch v1 Beautiful
Attachment #509933 - Flags: review?(ian) → review+
(In reply to comment #5) > Created attachment 509933 [details] [diff] [review] > patch v1 Passed try: http://tbpl.mozilla.org/?tree=MozillaTryfirstname.lastname@example.org&rev=0cc58f595787
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b12
Whiteboard: [hardblocker][has patch] → [hardblocker][has patch][qa-]
You need to log in before you can comment on or make changes to this bug.