<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/105059/">http://git.reviewboard.kde.org/r/105059/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for rekonq.</div>
<div>By Anton Kreuzkamp.</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This patch addes a mpris2-compliant dbus-interface to rekonq that allows controling html5-video and -audio elements from e.g. the nowplaying-plasmoid.
This is espacially useful together with youtube's html5-player and the plasmoid PlayControl (which allows to assign global shortcuts to mediaplayer-actions (so you can pause all mediaplayers with the same mediacontrol-keys))
With this patch rekonq aggregates all opened media elements and provides one interface to dbus (providing one interface per media element is not possible with mpris) that contains the data of
and sends commands recieved via dbus to the last active mediaplayer (that was last loaded or changed its playback-status most recently).
The communication to the media element happens via javascript.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Comprehensive tests with different html5 mediaplayers (audio and video) and different mpris-controllers. No problems anymore.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/CMakeLists.txt <span style="color: grey">(1180ad0)</span></li>
<li>src/application.h <span style="color: grey">(36114ae)</span></li>
<li>src/application.cpp <span style="color: grey">(ef6c208)</span></li>
<li>src/dbus/mpris2/Mpris2DBusHandler.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/dbus/mpris2/Mpris2DBusHandler.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/dbus/mpris2/org.mpris.MediaPlayer2.Player.xml <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/dbus/mpris2/org.mpris.MediaPlayer2.root.xml <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/webpage.h <span style="color: grey">(abc9f64)</span></li>
<li>src/webpage.cpp <span style="color: grey">(ce1151d)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/105059/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>