D16519: Don't call scrollTo every time there's a mouseMoveEvent, use a timer instead

[kezi]

kezik created this revision.
kezik added a reviewer: Okular.
kezik added a project: Okular.
Herald added a subscriber: okular-devel.
kezik requested review of this revision.

REVISION SUMMARY
  BUG: 399208
  
  A workaround to the fact that on some hardware configurations with a wacom tablet (or, in general, an input device that generates a lot of events), okular accumulates several second of input latency
  
  The timer is hardcoded to run at 60 fps, maybe that's not elegant
  
  I see that there's similar functionality in a pair of other places, but I wasn't able to recycle the code efficiently
  
  I tested this on a weak celeron n3450 laptop with no performance drawbacks
  I don't notice any additional stuttering or lag on my powerful desktop

REPOSITORY
  R223 Okular

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

AFFECTED FILES
  ui/pageview.cpp
  ui/pageview.h

To: kezik, #okular
Cc: okular-devel, #okular, ngraham, aacid
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20181029/23efa6e0/attachment.html>