Bug 19115 - time to open context menu on a cu depends on the cu's size
Summary: time to open context menu on a cu depends on the cu's size
Status: RESOLVED DUPLICATE of bug 19300
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 2.0   Edit
Hardware: PC Windows 2000
: P2 major (vote)
Target Milestone: ---   Edit
Assignee: Erich Gamma CLA
QA Contact:
URL:
Whiteboard:
Keywords: performance
Depends on:
Blocks:
 
Reported: 2002-06-04 12:05 EDT by Adam Kiezun CLA
Modified: 2002-06-05 12:26 EDT (History)
0 users

See Also:

Attachments
profile (8.10 KB, text/plain)
2002-06-05 08:39 EDT, Adam Kiezun CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Kiezun CLA 2002-06-04 12:05:03 EDT 
F2
.file of 6000 lines
.selecting the file and opening the context menu in the packages view 
takes a long time
Comment 1 Erich Gamma CLA 2002-06-04 12:22:14 EDT 
problem confirmed.
The menu is only slow the first time, when there are now changes then it is 
fast. After a change and pop-up it is slow again.

Suspect someone is doing a getElementAt which triggers a reconcile on pop-up.
getElementAt should only be executed during running and not during enablment 
checking.
Comment 2 Erich Gamma CLA 2002-06-04 12:54:24 EDT 
The History actions trigger a reconcile:
org.eclipse.jdt.internal.ui.compare.JavaHistoryEditorAction.update() line: 50

We have to switch to optimistic enabling for the history actions.
Comment 3 Adam Kiezun CLA 2002-06-05 05:51:42 EDT 
editor context menu is another problem (see bug 19275)
Comment 4 Adam Kiezun CLA 2002-06-05 08:39:07 EDT 
unbelievable as it sounds it's ObjectActionContributor.testName() that calls 
toString() on this compilation unit (for every action, i think)!
i'm attaching profile
Comment 5 Adam Kiezun CLA 2002-06-05 08:39:38 EDT 
Created attachment 1254 [details]
profile
Comment 6 Kai-Uwe Maetzel CLA 2002-06-05 12:26:09 EDT 
Workbench problem see 19300 

*** This bug has been marked as a duplicate of 19300 ***