Bug 389287 - [Performance] Opening multiple editors results in flashing toolbars
Summary: [Performance] Opening multiple editors results in flashing toolbars
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 4.3   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 4.3 M2   Edit
Assignee: Bogdan Gheorghe CLA
QA Contact:
URL:
Whiteboard:
Keywords: performance
Depends on:
Blocks:
 
Reported: 2012-09-11 12:52 EDT by Bogdan Gheorghe CLA
Modified: 2012-09-27 11:50 EDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Bogdan Gheorghe CLA 2012-09-11 12:52:01 EDT
The toolbars seem to get updated too often when opening multiple editors (flashing chevron).

We might be computing item size too many times (once when initially creating the item and again on the resize event).
Comment 1 John Arthorne CLA 2012-09-11 13:40:50 EDT
There seemed to be updates triggered by multiple separate calls: tool item creation, setImage, setText, and a couple of others. This showed up near the top of the hotspots list in a sampling profile run of opening/closing a single Java editor multiple times.
Comment 2 Bogdan Gheorghe CLA 2012-09-24 17:27:36 EDT
The flashing of toolbars occured on Win32 only and was a direct result of having to flush any outstanding paints to a toolbar before measuring its new size. This code has been there since 2001 (no one here remembered the history). Upon further investigation, we determined that it might not be needed on Windows 7.

But all of this became a moot point as we changed our approach to drawing CTabFolders to batch all the redraw and layout requests.

Fixed in master (and will be potentially put into 4.2.2 via Bug 389773 after running the code for a while).