> On July 29, 2013, 8:29 a.m., Andrea Diamantini wrote: > > Ship It! > > Andrea Diamantini wrote: > Fantastic work, Paul! Please, let me know if you need help to merge it!
Thank you :) About the merging, I don't think that I can push things to the kde git-repositories. I'm also not sure how exactly to merge the patches. I guess I should start reading a git tutorial... Anyway, it would be nice if you could either merge the patches or give me commit rights, so that I can push my changes (after I find out how it works). - Paul ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111712/#review36708 ----------------------------------------------------------- On July 26, 2013, 11:14 a.m., Paul Rohrbach wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111712/ > ----------------------------------------------------------- > > (Updated July 26, 2013, 11:14 a.m.) > > > Review request for rekonq. > > > Description > ------- > > Load the adblock settings in background. > > If you open rekonq and start typing immediately, there is a small lag (about > half a second). This is caused by the AdBlockManager::loadSettings function > reading the adblock rules. It is called with a one second delay to make the > startup faster. With this patch, the function is called in a separate thread > using QtConcurrent::run to eliminate the lag. > > To make this safe, the adblocker is enabled after the settings are loaded, > which means, that the first site loaded could be partially with ads. But this > is no change in behavior, as the adblocker in current master is disabled for > a whole second. > > The only times this blocks now is in situations, where it is necessary that > the settings are loaded (showSettings and addCustomRule), but I guess that no > normal user will use those in under a second after startup :). > > > Diffs > ----- > > src/adblock/adblockmanager.h 3d329ad > src/adblock/adblockmanager.cpp 51000a3 > > Diff: http://git.reviewboard.kde.org/r/111712/diff/ > > > Testing > ------- > > - compiles, loads settings > - fixes the lag > > > Thanks, > > Paul Rohrbach > >
_______________________________________________ rekonq mailing list [email protected] https://mail.kde.org/mailman/listinfo/rekonq
