Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

high cpu load after running all tests? #36

Open
MrX1980 opened this issue Jan 28, 2012 · 5 comments
Open

high cpu load after running all tests? #36

MrX1980 opened this issue Jan 28, 2012 · 5 comments

Comments

@MrX1980
Copy link

MrX1980 commented Jan 28, 2012

Hello

after running all tests (results are visible) I see still "high" cpu load.
Tested with Firefox Release, Beta, Aurora, Nightly; Opera Next; Google Canary and IE9.
Only IE9 seems to be ok.

On a single core cpu I have about 100% and on a six core cpu about 17%

Can you find a reason for that?

@ghost ghost assigned yvg Feb 1, 2012
@yvg
Copy link

yvg commented Feb 1, 2012

Thanks @MrX1980
It seems to happen in the event-abort.js test
By checking out the git log I find out that audio.removeAttribute('src'); was replaced by audio.src = '';
By rolling this back it seems to fix the issue.

If I remember correctly we made this change because IE was crashing.
In both cases there seems to be browsers bugs here there we should report.

@MrX1980 Can you try out my change and see if it results in 100% CPU use for you?

note: awpy.js:39 also contains audio.src = ''; but doesn't seem to affect the CPU at all…
@tsenart got an other idea?

@MrX1980
Copy link
Author

MrX1980 commented Feb 2, 2012

Hello Yves,
I have still 100% cpu usage with Firefox Nightly, Opera Next and Chrome Canary.
IE9 is working fine.

In the error console of Firefox I have many entries of:
Warning: Invalid URI. Load of media resource failed.
Source File: http://areweplayingyet.org/
Line: 0

With Google Chrome -> JavaScript Console -> Network:
http://areweplayingyet.herokuapp.com/sounds/short.mp3?[SessionID]
Status = canceled
Type = undefined
Time = pending

I hope this can help.

@yvg
Copy link

yvg commented Feb 2, 2012

Did you try my proposed change locally? I didn't update the live code.

@MrX1980
Copy link
Author

MrX1980 commented Feb 2, 2012

Oh, I have tried the live code.
How I can try the local code?

@yvg
Copy link

yvg commented Feb 3, 2012

As explained in the readme here: https://github.com/soundcloud/areweplayingyet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants