D7580: Support loading by stream and restoring state on reload

[David Faure]

dfaure accepted this revision.
dfaure added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> svgpart.cpp:191
> +    // we can't tell if caller has explicitely set xOffset/yOffset of OpenUrlArguments
> +    // so in case of same url we just assume a reload and ignore the OpenUrlArguments xOffset/yOffset
> +    if (!mHasExtendedRestoreArguments && (url() == mPreviousUrl)) {

When I reload in konqueror (with KHTML or WebEngine), the yOffset is preserved.
Looking at KonqView::restoreHistory this is because restoreState is only called when reload==false.

It might be a good idea for kate/kdevelop to do the same, they have the actual information of whether reload was pressed, while parts can only guess.

REPOSITORY
  R383 SVGPart

BRANCH
  supportstreamandreload

REVISION DETAIL
  https://phabricator.kde.org/D7580

To: kossebau, #frameworks, dfaure
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170922/b74f635a/attachment.html>