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

Chrome-Audio-EQ Disables Sound on tunein.com #28

Closed
nealvb opened this issue Apr 30, 2015 · 10 comments
Closed

Chrome-Audio-EQ Disables Sound on tunein.com #28

nealvb opened this issue Apr 30, 2015 · 10 comments
Assignees
Labels

Comments

@nealvb
Copy link

nealvb commented Apr 30, 2015

Hi suddenly around mid April 2015 I stopped getting Audio (tunin.com) on one of my PC's via chrome
after tinkering around found it works (sound comes through) in incognito (where the plugin is diabled).

@ejci ejci added the bug label Apr 30, 2015
@ejci ejci self-assigned this Apr 30, 2015
@ejci
Copy link
Owner

ejci commented Apr 30, 2015

I will check that. Thanks for letting me know

@nealvb
Copy link
Author

nealvb commented May 1, 2015

Cool
Thank you for awesome plugin
On 30 Apr 2015 15:18, "Miroslav Magda" [email protected] wrote:

I will check that. Thanks for letting me know


Reply to this email directly or view it on GitHub
#28 (comment).

@hierovision
Copy link

I'd like to add that I also lose sound with this plugin enabled while in https://clyp.it/ and the steam web store while watching videos.

@skylize
Copy link
Contributor

skylize commented Jul 13, 2015

Line numbers below are in reference to page.js at commit f56a7ef.

Some leads on fixing tunein.com, but haven't pinpointed it yet. It uses a stream url that looks like http://icy1.abacast.com/kplu-jazz24mp3-32 with no trailing slash and no file extension. When the extension is running, a forward slash and semicolon are somehow getting appended to the end causing a 404, so the above url gets transformed to http://icy1.abacast.com/kplu-jazz24mp3-32/;

On page load, the audio element has no source, so I tried conditionally adding crossorigin attribute (line 91) only if it doesn't exist and commenting out the line that resets the src (line 96). In that case src is only set by the app, instead of the extension, so the first time that if (target.src) matches at line 93, src is correct without the trailing characters. However it still gets mangled before attach() (function lines 74-122) is called again and before the audio can start playing. Line 96 is the only time we explicitly set an element's src so its apparently a side effect of something else.

@skylize
Copy link
Contributor

skylize commented Jul 13, 2015

Clyp.it is another issue altogether. With the extension disabled, if I just add crossorigin to the audio element from DevTools, a clip will continue playing till if finishes, but cannot be played a second time. Additionally, if the extension is enabled, but set to disabled for that site, it still breaks playback.

@ejci ejci closed this as completed in ed49d1f Jul 13, 2015
@ejci
Copy link
Owner

ejci commented Jul 13, 2015

I will release 0.3.2.1 version with fix

@skylize
Copy link
Contributor

skylize commented Jul 13, 2015

I think it's worth clarifying what was and was not fixed. The EQ still cannot actually be used on either tunein.com or on clyp.it. However fix ed49d1f makes the context menu option to "Disable for this domain" more reliable, so it's possible to use these sites without having to completely disable the extension.

@ejci
Copy link
Owner

ejci commented Jul 13, 2015

Yes you are right. Sorry for not being clear.

@Morteza-Rastgoo
Copy link

this problem still not fixed?

@nealvb
Copy link
Author

nealvb commented Oct 2, 2015

Hi just checked on tunein.com. the following happens:

*Open Chrome (Audio EQ currently disabled)
*Go to tunein.com and stream something -- all good
*Enable Audio EQ -- all good
*Try change the EQ -- audio is not effected
*Close and reopen Chrome (Audio EQ currently enabled)
*Go to tune.com and stream something -- get the following error message from tunein.com
"We Didn't Find Any Playable Streams"
*Disable Audio EQ -- still same error message
*Close and reopen chrome (Audio EQ currently disabled)
*Go to tune.com and stream something -- all good

Just check cliffcentral.com (another site I stream from) --ALL GOOD with Audio EQ
Though I see I have to disable or enable Audio EQ before I open a tab(site) that I'm going to stream from ...didn't realize that
So for me no, not yet fixed.
( In fact worse than before though I'm now running via chrome signed in where as before I was not)
Pity though :(
...as I really liked this extension.

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

No branches or pull requests

5 participants