Review Request 101850: Konqueror: Ask for session restore ONLY on plain startup (not for every window)

David Faure faure at kde.org
Mon Oct 27 20:44:13 GMT 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/101850/#review69218
-----------------------------------------------------------


Any plans to update the patch according to our discussion?

- David Faure


On July 4, 2011, 11:44 p.m., Marcel Partap wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/101850/
> -----------------------------------------------------------
> 
> (Updated July 4, 2011, 11:44 p.m.)
> 
> 
> Review request for KDE Base Apps and David Faure.
> 
> 
> Repository: kde-baseapps
> 
> 
> Description
> -------
> 
> This patch stops asking for session restore for EVERY new window (popups!) and does so ONLY when initial konqueror process is started, and called without URL. Because i didn't want to loose all my crashed sessions and the 'import crashed sessions as bookmarks' feature disappeared sometime, i had been clicking on 'ask again later' 42 bazillion times. Restoring all the crashed sessions always brought down the process (imho each session should restore as individual process, but that is another patch).
> 
> 
> Diffs
> -----
> 
>   konqueror/src/konqmain.cpp 218e708 
>   konqueror/src/konqmisc.cpp c46dc07 
> 
> Diff: https://git.reviewboard.kde.org/r/101850/diff/
> 
> 
> Testing
> -------
> 
> Tryed out and traced everything with GDB, does as says. Finally no huge crashed-session-parsing-delay for every stupid popup ^^
> 
> 
> Thanks,
> 
> Marcel Partap
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20141027/bad68f16/attachment.htm>


More information about the kde-core-devel mailing list