Issue metadata
Sign in to add a comment
|
Chrome/ium 53+ Plays Wrong Synth Notes
Reported by
devinuli...@gmail.com,
Dec 6 2016
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 6.1; rv:38.0) Gecko/20100101 Firefox/38.0 Steps to reproduce the problem: 1. Run a program that uses tone.js (such as https://walterbender.github.io/musicblocks) 2. With Music Blocks, click the rabbit to run the code (you may open console to see intended output) and this branch (https://github.com/pikurasa/musicblocks/tree/2016-11-21-wrongnotecheck) has debugging log output for tone.js. 3. About 75% of the plays will result in some kind of wrong note (sol4 mi4 sol4 should sound like "I love you" from Barney. Note: In the console, the intended output reported by the console for BOTH Music Blocks and tone.js match, however the note called by Chrome/ium 53+ is incorrect) This issue has been reproduced across the following operating systems: Windows (using latest Chrome), Mac (using latest Chrome), and GNU/Linux (using both Chrome and Chromium). Details: The versions of Chrome for Windows and Mac were not noted (other people's computers and no longer have access to that info), but we can safely assume they were "up-to-date". This issue does not occur with our tested versions of GNU/Linux (Fedora and Mint) that have an older version of Chrome/ium (52), but DOES occur with our tested versions of GNU/Linux that have newer versions of Chromium (53+). What is the expected behavior? I expect Chrome/ium to play the correct notes. It works correctly in FireFox family (just successfully tested with Abrowser 49.0.2, for example) What went wrong? It plays the wrong sound. Project bug reports that are related * https://github.com/walterbender/musicblocks/issues/295 * https://github.com/Tonejs/Tone.js/issues/181 Chrome/ium specific * Perhaps this is the specific bug for Chrome/ium (I am unsure) -- https://bugs.chromium.org/p/chromium/issues/detail?id=645776 Did this work before? Yes 52 Chrome version: <Copy from: 'about:version'> Channel: n/a OS Version: 6.1 (Windows 7, Windows Server 2008 R2) Flash Version: Flash is Evil Thanks! Music Blocks (SugarLabs) is a project participating in Google Code-In this year, which started last week (November, 2016). The sooner you fix this, the sooner we can recommend that the kids use "Chromium-family" as well as FireFox to accomplish their given tasks. Right now we are telling them to avoid Chrome/ium because it "plays wrong notes"...
,
Dec 8 2016
,
Dec 8 2016
I tried the repro case on Win10, Chrome 54, the current stable release. The notes play out correctly for me. The notes are the same as when I use Firefox to play. Works fine in Chrome canary (57). Could you try a newer version of Chrome?
,
Dec 9 2016
I am using Chromium Version 53.0.2785.143 Built on Ubuntu , running on Ubuntu 16.04 (64-bit), which is the version in the stable repos for Ubuntu. I just now ran an update, but the only Chromium related update is: chromium-browser-l10n Here are two links to videos I created of the issue: https://owncloud.libretools.com/index.php/s/xS42Pm96x3uVlhl https://owncloud.libretools.com/index.php/s/tWyXwAENYiu951M ^^Perhaps the second video would be more helpful. In the second video, I am using the tone.js code so that you can see the synth log side-by-side our log. The issue is that the both synth AND our software report the correct note, but the note that is played is the wrong note. You have to listen to notice the bug, hence the video. On another machine, I have version 54.0.2820.0 (64 bit) and I experience the same issue. As stated before, the issue was noticed on Windows, Mac, and GNU/Linux machines running Chrome/ium 53 and higher. If there is a newer stable version of Chromium that does not have the bug, would you please try to add it to the Ubuntu Repos? Thanks again! Devin
,
Dec 9 2016
Ok. Thanks for the videos. I do hear it, but you're also moving blocks around so I don't know what that means. I tried this with Chrome 55 (stable) on Ubuntu. I do hear the first note being played with the wrong pitch once in a while. I have been unable to reproduce this with Chrome beta or chrome dev. So some fixes maybe have come in between what you're using and Chrome beta, but without a smaller repro case that doesn't use tone.js, finding the underlying cause likely won't happen. Especially since I can't reproduce this with Chrome beta.
,
Dec 9 2016
> Ok. Thanks for the videos. I do hear it, but you're also moving blocks around so I don't know what that means. Moving the blocks just helped me isolate pitches. As you can see from the log, the intended output is unaffected. > I tried this with Chrome 55 (stable) on Ubuntu. I do hear the first note being played with the wrong pitch once in a while. I have been unable to reproduce this with Chrome beta or chrome dev. So some fixes maybe have come in between what you're using and Chrome beta... Do you mean that the newest versions after version 55 (stable) may have fixed this issue? Excuse my ignorance, but are these the newest beta versions? https://www.google.com/chrome/browser/beta.html?platform=linux https://www.google.com/chrome/browser/desktop/index.html?platform=linux&extra=devchannel > but without a smaller repro case that doesn't use tone.js, finding the underlying cause likely won't happen. Especially since I can't reproduce this with Chrome beta. "repro case" means "reproducible case"? I will try and test with latest versions. If I still experience the issue, I will try to experiment with other synths. Thanks!
,
Dec 9 2016
I assume the logs are just telling you what the output was supposed to be without actually measuring the actual output. Yes, those links are the correct ones for downloading beta and dev channel builds of chrome. Yes, "repro case" means reproducible case. Sorry for the jargon. Thanks for testing. It's much easier to find and fix bugs if the test case is small; having tone.js involved makes it much more difficult to do and I couldn't even find where you were creating the tones. (Because I don't know tone.js).
,
Jan 17 2017
,
Jan 25 2017
We do not seem to be experiencing the issue since we updated tone.js Please see https://github.com/walterbender/musicblocks/issues/295#issuecomment-275160656 and https://github.com/walterbender/musicblocks/commit/343c1b28d4013ef27240ab8a789379ab671c3a87 If this is the case, I apologize for raising the alarm. Thank you!
,
Jan 25 2017
devinulibarri@ Which version of Chrome did you test with?
,
Jan 25 2017
Version 54.0.2820.0 (64-bit), which is one of the versions that I had tested previously and reproduced the bug. I am asking for our team (SugarLabs) to test on various versions and report back to us. I will post their findings once they come in. I am especially curious about the results for Chrome/ium 53 - Chrome/ium 55 (stable).
,
Sep 13 2017
Based on c#9 and the fact that no new information has been given, I'm closing this bug as Fixed since it seems Chrome 54 has fixed it. If this issues shows up again, please re-open this issue or file a new one. Thanks! |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by dtapu...@chromium.org
, Dec 6 2016Components: -Blink Blink>WebAudio