Bug 105003 - Rotating a PNG image is quite slow
Summary: Rotating a PNG image is quite slow
Status: RESOLVED WORKSFORME
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
4.2.0.4 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: Writer-Images
  Show dependency treegraph
 
Reported: 2016-12-30 13:30 UTC by Telesto
Modified: 2018-03-25 11:08 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
Example file containing a PNG (1.15 MB, application/odt)
2016-12-30 13:31 UTC, Telesto
Details
Example file containing a JPG (691.69 KB, application/3dr)
2016-12-30 13:31 UTC, Telesto
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Telesto 2016-12-30 13:30:12 UTC
Description:
Rotating a PNG image is quite slow (and CPU intensive). Takes around 5 seconds.
Problems have intensified since LibO 4.2.0.4, because jpg images are converted to png before rotating. 



Steps to Reproduce:
1.Open attached file
2.Right click the image: Rotate -> Rotate 180 degrees

Actual Results:  
Rotating takes around 5 seconds

Expected Results:
Rotating should be instantaneous



Reproducible: Always

User Profile Reset: No

Additional Info:
Version: 5.4.0.0.alpha0+
Build ID: d0288a482a3dc0f50f535565e4c66a95bb140942
CPU Threads: 4; OS Version: Windows 6.19; UI Render: default; 
TinderBox: Win-x86@42, Branch:master, Time: 2016-12-26_23:25:18
Locale: nl-NL (nl_NL); Calc: CL

and in
Versie: 4.1.0.4 
Build ID: 89ea49ddacd9aa532507cbf852f2bb22b1ace28


User-Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:45.0) Gecko/20100101 Firefox/45.0
Comment 1 Telesto 2016-12-30 13:31:06 UTC
Created attachment 130030 [details]
Example file containing a PNG
Comment 2 Telesto 2016-12-30 13:31:36 UTC
Created attachment 130031 [details]
Example file containing a JPG
Comment 3 Jacques Guilleron 2017-01-02 10:02:03 UTC
Hi Telesto,

I reproduce this slow rotation (90°) with 
LO 5.4.0.0.alpha0+
Build ID: 5903235d57acb13d9d5286d23b443a01aeab9a3c
CPU Threads: 2; OS Version: Windows 6.1; UI Render: default; 
TinderBox: Win-x86@42, Branch:master, Time: 2016-12-19_00:08:35
Locale: fr-FR (fr_FR); Calc: CL

Same behaviour with:
LO 4.2.0.3 Build ID: c63c03decdf780d8fb80823950665b782ec9ecd0

I found your wish into 
LO 4.2.0.0.beta1 Build ID: f4ca7b35f580827ad2c69ea6d29f7c9b48ebbac7
where that was immediate.
OS: Windows 7 Home
Comment 4 Xisco Faulí 2017-01-25 20:36:05 UTC
Image rotation was introduced by commit df155415d8f46d884ba18e370e8028d020ba6f3b and going back to that point it already takes ~5 seconds, so this is not a regression at all.
Comment 5 QA Administrators 2018-01-26 03:27:08 UTC Comment hidden (obsolete)
Comment 6 Telesto 2018-03-25 11:08:24 UTC
No repro
Version: 6.1.0.0.alpha0+
Build ID: dd4f1b1bd31daf080dc0420524712dc244e539b5
CPU threads: 4; OS: Windows 6.3; UI render: default; 
TinderBox: Win-x86@42, Branch:master, Time: 2018-03-20_23:26:38
Locale: nl-NL (nl_NL); Calc: group