Comments on: ARGH at our unresponsive tab strip: setTimeout(foo, 0) can be very harmful

By: Raghavendra

Raghavendra — Sun, 22 Apr 2012 18:57:13 +0000

I commented on it here – https://bugzilla.mozilla.org/show_bug.cgi?id=743069#c14

By: kamulos

kamulos — Tue, 17 Apr 2012 17:58:35 +0000

@tglek: It does have a delay for me even if I disable hardware acceleration. I can reproduce it more easily when I switch to about:memory, click minimize memory and switch back. Because the built-in profiler only outputs hex-adresses for me all I can show now is the output of about:jank:

31 – c-JS::CallEventHandler

By: Manoj Mehta

Manoj Mehta — Tue, 17 Apr 2012 06:05:05 +0000

Looks like a reviewer has made that comment already. The changes are in good hands (not that I doubted it)

Manoj

By: tglek

tglek — Mon, 16 Apr 2012 17:07:31 +0000

@Manoh, please comment in bugs and refer to snippets of code in question. It’s not productive to say that here

kamulos, I can’t reproduce that. Have you tried turning off gfx acceleration?

By: Manoj Mehta

Manoj Mehta — Mon, 16 Apr 2012 04:27:16 +0000

Taras,

Glad to see this issue getting attention. From looking at the changes made to the locking code, it appears that locks are being introduced where they are not needed.

Thanks for providing the links to the bugs so we can follow along as fixes are posted.

Manoj

By: kamulos

kamulos — Sun, 15 Apr 2012 12:08:34 +0000

Maybe I have a different problem here. I am not sure. If I open http://www.escapistmagazine.com/videos/view/unskippable/1713-Deadly-Premonition-Part-1 , switch to a different tab, wait 30s and switch back it takes an exceptional large amount of time.

By: timeout

timeout — Sat, 14 Apr 2012 10:24:40 +0000

Well I’m glad that work is under way to make tab switching quicker,however I too wouldn’t dismiss the embarrassing delay when switching groups in Panorama/TabCandy as a non-primary ui issue,this feature has been with Firefox for a long time now,and,while undoubtedly useful and with a great potential,still to date doesn’t work as expected and promised from the beginning.

By: Dan

Dan — Fri, 13 Apr 2012 03:23:26 +0000

To be perfectly honest, I’d probably just make the tab thumbnailing code run via a setTimeout(funcname, 0)… then it will run after the normal tab stuff, problem solved! >_> lol

I code JavaScript for a living btw, haha >_>

By: tglek

tglek — Wed, 11 Apr 2012 16:36:45 +0000

@pd. There is a bug filed for turning off tab screenshots: http://bugzil.la/726347.

Rest assured, we’ll fix this particular tab thumbnail perf issue.

By: pd

pd — Wed, 11 Apr 2012 12:18:44 +0000

Firefox wasting resources on tab switching by taking a thumbnail is the last straw for me in terms of this new tab page malarky. I do not give a shit about a new tab page!! I never use the new tab button since I have open-new-tabs-from-* set up. Stuff this new tab crap slowing down my experience. Firefox does enough of that already! Please tell me how to turn off this unwanted thumbnail-creation-on-tab-click behavior.