1083203 - humble bundle snippet causes high cpu load

Status: RESOLVED FIXED

(Snippets Graveyard :: General, defect)

Description

[[:philipp]]

the current humble bundle snippet on about:home causes a rather high cpu load, even when the user hasn't interacted with it yet. 
can the content somehow be made static until a user has followed the lead to click on the content and actively plays with it?

Comment 1

[Osmose [:osmose, :mkelly]]

Thanks for the report! I also see a large spike in CPU.

jgruen, any idea of how difficult this change would be?

Comment 2

[John Gruen [:jgruen]]

Hey guys, thanks for bringing this up. The game developer did a bunch of optimization to reduce CPU overhead, but I'm seeing something similar now on about:home. Let me speak with him and see if we can't sort this out.

Comment 3

[John Gruen [:jgruen]]

Hey Mike, we have a fix for this on the dev server. Basically changed the code to suspend all drawing on the canvas until the user clicks. Seems to really reduce overhead. Just got it in if you want to help test it out. 

https://snippets.allizom.org/show/49/

Comment 4

[M.J. Kelly]

Flagging mkelly for this.

Comment 5

[John Gruen [:jgruen]]

oops, got my kellys crossed

Comment 6

[John Gruen [:jgruen]]

So I've tested this pretty broadly on OSX, Win7, Win8, and Ubuntu. Definitely improves performance on all platforms with no evident hiccups. 

https://github.com/mozilla/snippets/pull/14

Comment 7

[Bill Maggs (bmaggs)]

Mike, can you check and get it out there?

Comment 8

[Fabio Rios [:frios]]

Confirming update has been made.

Comment 9

[Osmose [:osmose, :mkelly]]

Yep, update has landed on prod, and the PR mentioned has been merged.

philipp: It may take up to 24 hours for your homepage to get the update, but if you're still seeing high CPU loads, you can force a refresh of the snippet code by:

- Opening the Web Console on about:home via Tools -> Web Developer -> Web Console
- Entering gSnippetsMap.clear() and hitting enter to run it in the web console.
- Clearing your entire cache using History -> Clear Recent History, setting the time to "Everything" checking _only_ the "Cache" box.
- Refreshing your about:home page.

Whether you wait 24 hours or you force a refresh, can you confirm if the CPU usage has dropped significantly for you? Thanks!

Comment 10

[Osmose [:osmose, :mkelly]]

(I'm leaving this unresolved in case we have other fixes we want to put out, will mark as resolved after a discussion tomorrow about what other fixes we want to make.)

Comment 11

[[:philipp]]

yes, the cpu usage has now dropped noticeably. however on an older machine it still takes 5-13% just sitting there, so there is still a bit of room for improvement.

Comment 12

[Osmose [:osmose, :mkelly]]

(In reply to philipp from comment #11)
> yes, the cpu usage has now dropped noticeably. however on an older machine
> it still takes 5-13% just sitting there, so there is still a bit of room for
> improvement.

Cool, thanks for confirming the improvement. :D

Comment 13

[Bill Maggs (bmaggs)]

Great to hear. We will be linking to the gamelet in social media even after we move to other snippets on about:home, so any further improvement will be appreciated.