<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/112076/">http://git.reviewboard.kde.org/r/112076/</a>
     </td>
    </tr>
   </table>
   <br />









<table bgcolor="#f0f0f0" cellpadding="5" cellspacing="5" style="border: 1px solid #c0c0c0; margin-bottom: 10px">
 <tr>
  <td><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/08/30/dragoninhibitreworknomainwindow.diff" style="color: black; font-weight: bold; font-size: 9pt;">stateChange and mainWindow-less power patch</a></td>
 </tr>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">stopSuppressingScreenPowerManagement also inhibits the screensaver since 4.8, so the need to suppress the screensaver only with KNotificationRestrictions for the moment isn't there.(?)</pre>
<br />





<p>- James</p>


<br />
<p>On August 30th, 2013, 11:05 a.m. UTC, James Smith wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for KDE Multimedia and Harald Sitter.</div>
<div>By James Smith.</div>


<p style="color: grey;"><i>Updated Aug. 30, 2013, 11:05 a.m.</i></p>






<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;">Fixes an issue where playing audio content keeps the screen awake. Video content is supposed to, but with the new play URL button, dragon should allow the monitor to turn off.</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;">Compile, run-test</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/app/mainWindow.h <span style="color: grey">(51ef72c)</span></li>

 <li>src/app/mainWindow.cpp <span style="color: grey">(f850820)</span></li>

 <li>src/app/stateChange.cpp <span style="color: grey">(0edde72)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/112076/diff/" style="margin-left: 3em;">View Diff</a></p>



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>

<ul>

 <li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/08/30/dragoninhibitreworknomainwindow.diff">stateChange and mainWindow-less power patch</a></li>

</ul>





  </td>
 </tr>
</table>








  </div>
 </body>
</html>