Eclipse Community Forums: Eclipse 4 » Expensive updates of ToolItems

Home » Eclipse Projects » Eclipse 4 » Expensive updates of ToolItems

Expensive updates of ToolItems [message #879949]

Fri, 01 June 2012 03:31

Eclipse User

Hi,

while working with e4 I realized, that org.eclipse.e4.ui.workbench.renderers.swt.HandledContributionItem.updateItemEnablement() is called about once a second for every ToolItem in my MainToolbar. This leads to HandledContributionItem.updateIcons(). I think this call is needed here to switch between enabled/disabled icons. But even if there is no change the icon bytes are requested via iconURI and a new SWT Image is created everytime.
A similar problem exists for the labels and tooltips, which are newly translated via TranslationService on every updateItemEnablement-call.

Is this a bug? Or required by some reason I don't see yet?

Marko

Re: Expensive updates of ToolItems [message #879956 is a reply to message #879949]

Fri, 01 June 2012 03:45

Eclipse User

File a bugzilla - this update should be as lightweight as possible.

Tom

Am 01.06.12 09:31, schrieb Marko Herchet:
> Hi,
>
> while working with e4 I realized, that
> org.eclipse.e4.ui.workbench.renderers.swt.HandledContributionItem.updateItemEnablement()
> is called about once a second for every ToolItem in my MainToolbar. This
> leads to HandledContributionItem.updateIcons(). I think this call is
> needed here to switch between enabled/disabled icons. But even if there
> is no change the icon bytes are requested via iconURI and a new SWT
> Image is created everytime.
> A similar problem exists for the labels and tooltips, which are newly
> translated via TranslationService on every updateItemEnablement-call.
>
> Is this a bug? Or required by some reason I don't see yet?
>
> Marko

Re: Expensive updates of ToolItems [message #880062 is a reply to message #879956]

Fri, 01 June 2012 06:54

Eclipse User

Well the image is not recreated it's the same instance all the time
because it is retrieved from the resource manager.

Tom

Am 01.06.12 09:45, schrieb Tom Schindl:
> File a bugzilla - this update should be as lightweight as possible.
>
> Tom
>
> Am 01.06.12 09:31, schrieb Marko Herchet:
>> Hi,
>>
>> while working with e4 I realized, that
>> org.eclipse.e4.ui.workbench.renderers.swt.HandledContributionItem.updateItemEnablement()
>> is called about once a second for every ToolItem in my MainToolbar. This
>> leads to HandledContributionItem.updateIcons(). I think this call is
>> needed here to switch between enabled/disabled icons. But even if there
>> is no change the icon bytes are requested via iconURI and a new SWT
>> Image is created everytime.
>> A similar problem exists for the labels and tooltips, which are newly
>> translated via TranslationService on every updateItemEnablement-call.
>>
>> Is this a bug? Or required by some reason I don't see yet?
>>
>> Marko
>

Re: Expensive updates of ToolItems [message #880171 is a reply to message #880062]

Fri, 01 June 2012 10:52

Eclipse User

Ok Tom you are right. The URLImageDescriptor is created on every call but createImage() is never called because a cached image is already maped for the URLImageDescriptor.

Thanks for your help

Re: Expensive updates of ToolItems [message #901822 is a reply to message #879956]

Tue, 14 August 2012 13:17

Eclipse User

Hi Tom,
Could you explain why this updateItemEnablement is needed? It is being called on a timer every 400ms and is expensive just because it's running on the UI thread. Is this covering up other bugs? Is it possible that you could do less on the UI thread?

I can enter a bug for this issue, I was just hoping for some more explanation. I still think it's a problem even though the images are not really being recreated every time. At the least the updateIcons could be performance tweaked, in the best case scenario it doesn't need to be called on the UI thread or called constantly.

I have attached a snapshot from jvisualvm showing a few seconds of time doing nothing in a Java perspective with open java editors.

Thanks,
Karen

Attachment: snapshot-8_14_2012-ToolItemUpdateTimer.nps
(Size: 115.10KB, Downloaded 256 times)

Re: Expensive updates of ToolItems [message #901860 is a reply to message #901822]

Tue, 14 August 2012 15:18

Eclipse User

This issue is already solved with a better approach and will be included in the next builds. Would be great if you could participate in testing it.See https://bugs.eclipse.org/bugs/show_bug.cgi?id=386070 and the related RGB leak bug.

Re: Expensive updates of ToolItems [message #901884 is a reply to message #901822]

Tue, 14 August 2012 17:27

Eclipse User

Hi Karen,

IMHO we should switch from the timerExec to a RunAndTrack approach but
I'm not too familiar with the code.

For my JavaFX rendering I'm for now going the RAT-Route to see how that
works out.

Tom

Am 14.08.12 19:17, schrieb Karen Butzke:
> Hi Tom,
> Could you explain why this updateItemEnablement is needed? It is being called on a timer every 400ms and is expensive just because it's running on the UI thread. Is this covering up other bugs? Is it possible that you could do less on the UI thread?
>
> I can enter a bug for this issue, I was just hoping for some more explanation. I still think it's a problem even though the images are not really being recreated every time. At the least the updateIcons could be performance tweaked, in the best case scenario it doesn't need to be called on the UI thread or called constantly.
>
> I have attached a snapshot from jvisualvm showing a few seconds of time doing nothing in a Java perspective with open java editors.
>
> Thanks,
> Karen
>

Re: Expensive updates of ToolItems [message #902025 is a reply to message #901860]

Wed, 15 August 2012 10:19

Eclipse User

Thanks, i didn't see the bug since I was only searching for open ones! Performance looks much better now, thanks!

I see one issue with the main toolbar, which I don't know if it was caused by this bug or not. The separators are gone now and the toolbar sections are spread out. I have attached 2 screen shots showing the problem

Attachment: 4.2ToolBar.jpg
(Size: 7.22KB, Downloaded 370 times)
Attachment: 4.2M20120809-1200ToolBar.jpg
(Size: 8.30KB, Downloaded 372 times)

Re: Expensive updates of ToolItems [message #902028 is a reply to message #902025]

Wed, 15 August 2012 10:32

Eclipse User

No the timer thing has no connection with the toolbar glitch. That's because the commit on bug https://bugs.eclipse.org/bugs/show_bug.cgi?id=340695 and it should be fixed on the next I-build i guess.

Re: Expensive updates of ToolItems [message #912474 is a reply to message #902028]

Thu, 13 September 2012 12:32

Eclipse User

The toolbar layout is still broken in SR1 RC2. See my above screenshots. This is a regression from the 4.2 release to the SR1 RC2. https://bugs.eclipse.org/bugs/show_bug.cgi?id=340695 has not been fixed.

Re: Expensive updates of ToolItems [message #1671125 is a reply to message #912474]

Fri, 13 March 2015 09:24

Eclipse User

I was wondering whether this bug in the main toolbar (gone separators and spread out toolbars) is finally fixed. I doubt, since I still experience it in our RCP application built with eclipse luna 4.4.1.
The main toolbar still looks exactly the same as the one in the screenshot posted by Karen.
So what's the state of this bug?
See attached screenshot of our main toolbar

Attachment: brokenToolbars.png
(Size: 12.55KB, Downloaded 128 times)

Previous Topic:	Can we select and double icons using E4 CSS?
Next Topic:	DynamicMenuContribution and IEclipseContext

Goto Forum:

-=] Back to Top [=-

Current Time: Sat Jul 26 06:51:20 EDT 2025

.:: Contact :: Home ::.

Breadcrumbs

Sign up to our Newsletter