<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, fvogt, cblack.<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/D28614">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>There is unfortunately no dedicated signal for this (the process API is for dev builds only), so we can only do:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">When we detect a tab becoming inaudible, check if it's gone now, and signal player gone. This will catch the case of a tab crashing while playing</li>
<li class="remarkup-list-item">When sending a command, check if it failed, and signal player gone. Instead of doing nothing, the player will disappear. Not really better UX right there but at least the user will go "huh?" rather than "why is it not working?"</li>
</ul>

<p>There's also no error codes on the error object, just sentences...</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Started playing a tab in chrome, killed the process, had the player gone after a few seconds. This heuristic does not work in Firefox where instead I get a navigation to about:blank and back to the original page...</li>
<li class="remarkup-list-item">Started playing a tab in chrome, paused it, killed the process. Nothing changed. I then clicked Play on media controller, and the player was gone. This also works in Firefox where the error message is identical.</li>
</ul></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/D28614">https://phabricator.kde.org/D28614</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>extension/extension-mpris.js</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, fvogt, cblack<br /><strong>Cc: </strong>plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>