~jamesh/thumbnailer/qml-module-package-name

Viewing all changes in revision 268.

We now handle unexpected network errors correctly, re-trying only after the wait time has expired. Added logging for requests with invalid size to the service-side log, so we don't have look through all the application logs to spot when this happens. The invalid requests are still logged in the application log, but are now adjusted by the service. Added correct error handling when we can't construct an ImageData instance and get an exception. This now correctly creates a failure cache entry. Fixed incorrect handling of requests for larger than the 1920-pixel limit. (The implementation incorrectly relied on invalidly-sized requests.) Disabled failure caching for failures from vs-thumb. If vs-thumb fails, it may work again next time. But, if we cache the failure, it won't manage to create a thumbnail again for the same image until the failure falls out of the cache (which is likely never). Added missing test fixture to slow-vs-thumb test, so the result doesn't depend on the actions of earlier tests or what's in the default cache location. Fixed tests to no longer use invalid QSizes. Changed (0,0) size semantics to limit returned image to 1920 bounding box.

Approved by PS Jenkins bot, James Henstridge.

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: