Review Request 125877: Fix kdeplatformtheme_unittest after last commit

David Rosca nowrep at gmail.com
Sat Nov 7 12:10:17 UTC 2015


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

(Updated Nov. 7, 2015, 12:10 p.m.)


Review request for KDE Frameworks.


Repository: frameworkintegration


Description (updated)
-------

Use Q_CONSTRUCTOR_FUNCTION for prepareEnvironment, so the QApplication's platform theme is using the test paths.

This also fixes updating the WheelScrollLines when the settings change, because QApplication is only using theme hint on startup. If the value changes, we need to call QApplication::setWheelScrollLines instead of just updating the theme hint.


Diffs (updated)
-----

  autotests/kdeplatformtheme_unittest.cpp f660ffd 
  src/platformtheme/khintssettings.cpp 14e1d43 

Diff: https://git.reviewboard.kde.org/r/125877/diff/


Testing (updated)
-------

Tests pass


Thanks,

David Rosca

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151107/897ba9eb/attachment.html>


More information about the Kde-frameworks-devel mailing list