> On Sept. 25, 2012, 1:27 p.m., Andrea Diamantini wrote: > > Ship It!
Can you merge it on your own or have I to take care about? - Andrea ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106561/#review19411 ----------------------------------------------------------- On Sept. 25, 2012, 4:46 a.m., David Narváez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106561/ > ----------------------------------------------------------- > > (Updated Sept. 25, 2012, 4:46 a.m.) > > > Review request for rekonq. > > > Description > ------- > > Check if the URL of the current tab is even valid before checking its > protocol. This fixes a bug where a new untitled tab is created on each window > but the first one when restoring from a crash (because the initial tab of all > windows but the first one have empty URLs). > > > Diffs > ----- > > src/sessionmanager.cpp d5bc1ff > > Diff: http://git.reviewboard.kde.org/r/106561/diff/ > > > Testing > ------- > > 1. Open multiple windows in rekonq > 2. Issue killall rekonq from a console > 3. Open rekonq again > 4. Select Yes on the restore a crashed session bar > > Before this patch, all windows but the first one would have an untitled empty > tab. After this patch the windows are restored just as they were when rekonq > crashed. > > > Thanks, > > David Narváez > >
_______________________________________________ rekonq mailing list [email protected] https://mail.kde.org/mailman/listinfo/rekonq
