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





 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">How does this relate to the call to
     mainWindow->resetAutoSaveSettings(); // Don't autosave
in KonqMainWindow::slotCreateNewWindow? It's kind of the same idea. Well, in fact no, it's better ;)

You're only setting "save window size" to false, while "resetAutoSaveSettings" means "don't autosave anything, not even toolbar placement or anything".

I guess what this means is: if the new bool is only set in slotCreateNewWindow then the existing resetAutoSaveSettings is fine, otherwise it needs to be done elsewhere too.
The code in setProfileConfig is about loading the default settings (from the profile), but I guess there isn't much to load for "popup windows without toolbars" indeed. (e.g. if someone had the sidebar enabled in the profile, it should probably not show up in such windows). So maybe rather a if() in that method.</pre>
 <br />







<p>- David</p>


<br />
<p>On August 3rd, 2012, 9:02 p.m., Dawit Alemayehu 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 KDE Base Apps and David Faure.</div>
<div>By Dawit Alemayehu.</div>


<p style="color: grey;"><i>Updated Aug. 3, 2012, 9:02 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;">Now that the special window popup scenario has been dealt with, the attached one liner attempts to resolve the issue of the size of such poup windows being saved whenver "setAutoSaveSettings" is invoked.</pre>
  </td>
 </tr>
</table>




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


 <a href="http://bugs.kde.org/show_bug.cgi?id=200907">200907</a>


</div>


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

 <li>konqueror/src/konqmainwindow.cpp <span style="color: grey">(6faba58)</span></li>

</ul>

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




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








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