<table><tr><td style="">broulik updated this revision to Diff 63618.<br />broulik retitled this revision from "Handle media session callbacks even when the requested one isn't supported by the browser" to "Handle media session callbacks before passing it on to the browser".<br />broulik edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-p7yjhepjkt3gogn/">(Show Details)</a><br />broulik added a comment.
</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/D23090">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Register ourselves before passing on to the browser which may throw</li>
</ul></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">The Media Session API standard evolves quite fast, so there's new actions that we might be able to support that the browser itself doesn't know yet, for instance the "stop" action.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">By check</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">By register</span>ing the <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">values</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">callback to</span> ourselves <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">and catch</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">before call</span>ing <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">an exception by</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">into</span> the browser, we're still able to handle <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">them</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">it</span> in the extensio<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">n</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">n even if not supported by the browser</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/D23090?vs=63532&id=63618">https://phabricator.kde.org/D23090?vs=63532&id=63618</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23090">https://phabricator.kde.org/D23090</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, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>