Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 337947

Summary: Write in source is very slow
Product: z_Archived Reporter: Stanislav Nichev <stanislav.nichev>
Component: WTP IncubatorAssignee: Dimitar Tenev <dimitar.tenev>
Status: NEW --- QA Contact: Dimitar Tenev <dimitar.tenev>
Severity: normal    
Priority: P3 CC: dimitar.donchev
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows Vista   
Whiteboard:
Attachments:
Description Flags
fix of the performance issue
none
fix for the test
none
adding test ensuring that the UI update is run in async mode none

Description Stanislav Nichev CLA 2011-02-23 05:01:43 EST
Build Identifier: 

Hi Colleagues,                                                      
    If you open an big WSDL and try to write in souce page, you will
see that this is really hard because the SIE is very slow.          
                                                                    
Regards,                                                            
Tzvetan Gaydev                                                      
                                                                    
    Memo                                                            
    12.11.2010 09:57:05      Stanislav Nichev                       
Issue update: live validation is performed on each start attribute, 
start tag, start value edit, etc...                                 
                                                                    
Regards,                                                            
Stanislav                                                           


Reproducible: Always
Comment 1 Stanislav Nichev CLA 2011-08-14 01:10:39 EDT
Hi Guys,

I've done some investigation of the issue.

It turned out that the problem is in the ValidationListener and more specifically:
resetPagesFormTitle() and updatePagesFormTitle() (ui part) methods.

I'm currently thinking of a redesign of the editor title update.

Regards,
Stanislav
Comment 2 Stanislav Nichev CLA 2011-08-14 02:36:24 EDT
Created attachment 201471 [details]
fix of the performance issue

Hi Guys,

I've prepared a fix for the performance issue when writing in source.

The problem was in the UI update of the validation messages. 
I've put this logic inside async runnable and it did the job.

Need to attach fix for one test.

Regards,
Stanislav
Comment 3 Stanislav Nichev CLA 2011-08-14 02:40:26 EDT
Created attachment 201472 [details]
fix for the test

Hi Guys,

I've added a "runAsyncUiUpdate" flag to the ValidationListener. If it is "true" - executes Display.asyncExec().

This flag will always be "true" for production, but it will be "false" for certain tests.
The benefits of this solution are more than the drawbacks, so I'll be happy to leave it as it is.

Regards,
Stanislav
Comment 4 Stanislav Nichev CLA 2011-08-14 02:52:46 EDT
Created attachment 201473 [details]
adding test ensuring that the UI update is run in async mode
Comment 5 Stanislav Nichev CLA 2011-08-14 02:55:06 EDT
Hi Guys,

I've prepared some patches for this issue.
Patches should be applied in the order they were created.

Regards,
Stanislav