Resending, since list admins haven't been able to approve the original post and we probably need to start reviewing this soon before 4.7.1.<br><br>David E. Narváez<br><br><div class="gmail_quote">---------- Forwarded message ----------<br>

From: <b class="gmail_sendername">David Narváez</b> <span dir="ltr"><<a href="mailto:david.narvaez@computer.org">david.narvaez@computer.org</a>></span><br>Date: Sun, Aug 28, 2011 at 9:14 PM<br>Subject: Review Request: Unify reloading methods in reKonq<br>

To: David Narváez <<a href="mailto:david.narvaez@computer.org">david.narvaez@computer.org</a>>, rekonq <<a href="mailto:rekonq@kde.org">rekonq@kde.org</a>><br><br><br>



 <div>
  <div style="font-family:Verdana, Arial, Helvetica, Sans-Serif">
   <table style="border:1px #c9c399 solid" cellpadding="8" width="100%" bgcolor="#f9f3c9">
    <tbody><tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="http://git.reviewboard.kde.org/r/102481/" target="_blank">http://git.reviewboard.kde.org/r/102481/</a>
     </td>
    </tr>
   </tbody></table>
   <br>


<table style="background-repeat:repeat-x;border:1px black solid" cellpadding="8" cellspacing="0" width="100%" bgcolor="#fefadf">
 <tbody><tr>
  <td>

<div>Review request for rekonq.</div>
<div>By David Narváez.</div>





<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Description </h1>
<table style="border:1px solid #b8b5a0" cellpadding="10" cellspacing="0" width="100%" bgcolor="#ffffff">
 <tbody><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">There are currently two families of methods handling reloads: one that uses QWebView's reload method and one that does more sophisticated handling of the about: scheme. This, apart from being an issue on its own, produces a bug where reloading a tab from the Reload Tab and Reload All Tabs tab context menu items cannot reload failed web pages while reloading it from the Reload button in the toolbar can.

The proposed patch modifies all 3 reloading methods (including one slot) by adding a webReload(WebTab *) slot that will ultimately handle all reloads, and modifying the workflow of the other 3 methods in the following way:

- webReload() -> webReload(WebTab *)
- reloadTab(int) -> webReload(WebTab *)
- reloadAllTabs() -> reloadTab(int) -> webReload(WebTab *)</pre>
  </td>
 </tr>
</tbody></table>


<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Testing </h1>
<table style="border:1px solid #b8b5a0" cellpadding="10" cellspacing="0" width="100%" bgcolor="#ffffff">
 <tbody><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">To reproduced the bug mentioned above:

1. Disconnect from the internet (I disable networking in my laptop), open rekonq, open 3 tabs and access any web page in those three tabs until you have all three tabs displaying error pages (Host Not Found).
2. Reconnect, and right click any tab, then do Reload Tab and Reload All Tabs, nothing should happen
3. Go to any tab and click the Reload button from the toolbar, you should be able to reload the web page

After applying the patch, you should be able to reload any tab from the tab context menu or from the toolbar, and also reloading all failed tabs from the Reload All Tabs item in the tab context menu.</pre>
  </td>
 </tr>
</tbody></table>




<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Diffs </h1>
<ul style="margin-left:3em;padding-left:0">

 <li>src/mainview.h <span style="color:grey">(ede1d62)</span></li>

 <li>src/mainview.cpp <span style="color:grey">(b573894)</span></li>

</ul>

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




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




  </div>
 </div>


</div><br>