<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, fvogt.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />broulik requested review of this revision.
</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/D20726">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Since Chrome 73 it also supports Media Session also on the desktop.<br />
The calls are re-arranged since <tt style="background: #ebebeb; font-size: 13px;">setActionHandler</tt> throws if passed an invalid action and in this case we don't want to register the callback either.<br />
Also, there's a new "skipad" action for a "Skip Ad" overlay in picture-in-picture mode [1] for which to work we need to call the native browser APIs.<br />
Also, since <tt style="background: #ebebeb; font-size: 13px;">MediaMetadata</tt> is a proper class and not an <tt style="background: #ebebeb; font-size: 13px;">Object</tt>, iterate its prototype rather than just stringifying it and adjust our shim to do the same.</p>

<p>[1] <a href="https://www.chromestatus.com/feature/4749278882824192" class="remarkup-link" target="_blank" rel="noreferrer">https://www.chromestatus.com/feature/4749278882824192</a></p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Tried this example [1] in Chrome, now get a "Skip ad" button in the PIP overlay.<br />
Firefox still works as before</p>

<p>[1] <a href="https://googlechrome.github.io/samples/picture-in-picture/skip-ad.html" class="remarkup-link" target="_blank" rel="noreferrer">https://googlechrome.github.io/samples/picture-in-picture/skip-ad.html</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R856 Plasma Browser Integration</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20726">https://phabricator.kde.org/D20726</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, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>