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








<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On July 21st, 2011, 12:30 p.m., <b>Pino Toscano</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  



<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
 <thead>
  <tr>
   <th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
    <a href="http://git.reviewboard.kde.org/r/101925/diff/1/?file=26754#file26754line795" style="color: black; font-weight: bold; text-decoration: underline;">src/webpage.cpp</a>
    <span style="font-weight: normal;">

     (Diff revision 1)

    </span>
   </th>
  </tr>
 </thead>

 <tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
  <tr>

   <td colspan="2"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
   <td colspan="2"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">bool WebPage::hasSslValid()</pre></td>

  </tr>
 </tbody>




 
 



 <tbody>

  <tr>
    <th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
    <th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">772</font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="n">Q_FOREACH</span><span class="p">(</span><span class="k">const</span> <span class="n">QSslCertificate</span> <span class="o">&amp;</span><span class="n">cert</span><span class="p">,</span> <span class="n">certList</span><span class="p">)</span></pre></td>
  </tr>

  <tr>
    <th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
    <th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">773</font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="p">{</span></pre></td>
  </tr>

  <tr>
    <th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
    <th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">774</font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="n">v</span> <span class="o">&amp;=</span> <span class="n">cert</span><span class="p">.</span><span class="n">isValid</span><span class="p">();</span></pre></td>
  </tr>

  <tr>
    <th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
    <th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">775</font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="p">}</span></pre></td>
  </tr>

  <tr>
    <th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
    <th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">776</font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="k">return</span> <span class="n">v</span><span class="p">;</span></pre></td>
  </tr>

 </tbody>

</table>

  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Just exit at the first invalid certificate:
  for (cert, ...) { if (!cert.isValid()) return false; }
  return true;</pre>
 </blockquote>





</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Actually this code is totally bogus. As before, this is merely checking the dates and blacklist status.</pre>
<br />




<p>- Richard J.</p>


<br />
<p>On July 11th, 2011, 10:21 p.m., Andrea Diamantini wrote:</p>






<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 Andrea Diamantini.</div>


<p style="color: grey;"><i>Updated July 11, 2011, 10:21 p.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;">The patch, available on the remote repo in the branch called &quot;SSL_Dialogs_Improvements&quot;,  provides 2 new GUIs, similar to the ones provided by Google Chrome, to notify users about SSL infos.
The patch allows also to export the certificate in binary form. Further upgrades in the SSL management have to be done in kdelibs.

&quot;Side&quot; change to render the same informations Chrom* does is about history: upgraded HISTORY_VERSION and added a firstDateTimeVisit entry in the HistoryTime struct.</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;">Tested against the following sites:
- https://localhost, with an untrusted and expired certificate
- https://encrypted.google.com
- https://paypal.com
- https://sites.nea.org/JoinNea/

In all these sites, rekonq behaves similar to what Chrom* does.</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">(f623adf)</span></li>

 <li>src/history/historymanager.h <span style="color: grey">(8650bd1)</span></li>

 <li>src/history/historymanager.cpp <span style="color: grey">(8113add)</span></li>

 <li>src/history/historymodels.h <span style="color: grey">(8cb1a5d)</span></li>

 <li>src/history/historymodels.cpp <span style="color: grey">(2cab8ef)</span></li>

 <li>src/sslinfo.ui <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/sslinfodialog.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/sslinfodialog.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/sslinfodialog_p.h <span style="color: grey">(72f1679)</span></li>

 <li>src/urlbar/sslwidget.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/urlbar/sslwidget.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/urlbar/urlbar.cpp <span style="color: grey">(078dc6a)</span></li>

 <li>src/webpage.h <span style="color: grey">(09977bf)</span></li>

 <li>src/webpage.cpp <span style="color: grey">(9499d6f)</span></li>

</ul>

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



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

<div>

 <a href="http://git.reviewboard.kde.org/r/101925/s/196/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/07/11/02_ssl_widget_400x100.png" style="border: 1px black solid;" alt="SSL Widget" /></a>

 <a href="http://git.reviewboard.kde.org/r/101925/s/197/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/07/11/03_ssl_dialog_400x100.png" style="border: 1px black solid;" alt="SSL info Dialog" /></a>

</div>


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








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