<table><tr><td style="">broulik updated this revision to Diff 69545.<br />broulik retitled this revision from "Apply autoplay prevention evasion also to document.createElement" to "Fixup new Audio() constructor and apply autoplay prevention evasion to document.createElement".<br />broulik edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-iudjwb52klw7g44/">(Show Details)</a><br />broulik edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-btr6dril2f7rwg2/">(Show Details)</a><br />broulik added a comment.<br />This revision is now accepted and ready to land.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D24870">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Only apply hack on playback</li>
<li class="remarkup-list-item">Use better syntax</li>
<li class="remarkup-list-item">More testing</li>
</ul></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">It affects audio player created this way the same as a `new Audio()`</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">The arguments apparently weren't properly forwarded to the constructor, breaking e.g. `new Audio("foo")` calls. This uses the much nicer spread syntax introduced in ES6.<br />
Also, `document.createElement("audio")` is affected in the same way as `new Audio()`<br />
The evasion is now only done when the player actually starts playing, not immediately when it is created, since we only care about playing players.</span><br />
<br />
BUG: 411742</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Google translate still works</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">The following were tested, play sound fine now (or still work), and get media controls if the played sound is longer than the minimum threshold (8 seconds):<br />
(Mostly aggregated from past p-b-i Bug reports about media controls)<br />
* Amazon Music<br />
* Spotify Website<br />
* Google Translate: https://translate.google.com/#view=home&op=translate&sl=auto&tl=de&text=take%20measures%20to%20check%20the%20quality%2C%20performance%2C%20or%20reliability%20of%20(something)%2C%20especially%20before%20putting%20it%20into%20widespread%20use%20or%20practice.<br />
* Cambridge dictionary: https://dictionary.cambridge.org/dictionary/english/example<br />
* Forvo.com https://forvo.com/search/test/<br />
* Cinemix radio stream https://www.radio.net/s/cinemix</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Cambridge dictionary works now</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">* Player FM</span> https://<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">dictionary.cambridge.org/dictionary/english/example</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">player.fm/series/der-nintendo-podcast</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Forvo.com works now</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">* This simple JS Fiddle done by Frederick Zhang in the bug report: https://jsfiddle.net/h78m6r0q/</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R856 Plasma Browser Integration</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D24870?vs=68565&id=69545">https://phabricator.kde.org/D24870?vs=68565&id=69545</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24870">https://phabricator.kde.org/D24870</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>extension/content-script.js</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, fvogt<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>