Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 31107 - CTabFolder and CCombo - too many sheels from tooltip shell
Summary: CTabFolder and CCombo - too many sheels from tooltip shell
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 2.1   Edit
Hardware: PC Windows XP
: P2 normal (vote)
Target Milestone: ---   Edit
Assignee: Veronika Irvine CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-02-06 11:55 EST by Eduardo Pereira CLA
Modified: 2003-11-03 10:08 EST (History)
0 users

See Also:

Attachments
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 Eduardo Pereira CLA 2003-02-06 11:55:06 EST 
Build id: 200302050800

From Nick:

I'm seeing many shells when I trace through BusyIndicator's call to 
Display.getShells().
There's one shell for the workbench window, then 5 children.  There's also 
another top-level shell that's not for a workbench window.  
None of the shells, except for the one for the workbench window, have any 
getData() back-refs.  All their handles are non-zero, so they're not disposed.
Comment 1 Eduardo Pereira CLA 2003-02-13 16:28:35 EST 
These extra shells are create in 
   CTabFolder.<init>(Composite, int) line: 186
   CCombo.<init>(Composite, int) line: 79

Moving to SWT. Maybe they could be lazy created. Or created and disposed as 
needed. Why do we need a shell to show a tooltip that may never popup or a 
combo that may not be used.

Moving to SWT.
Comment 2 Veronika Irvine CLA 2003-06-23 13:59:18 EDT 
CCombo does not have fake tooltips.  The extra shell in CCOmbo is from the 
popup list.  This can not be removed because the list is required in much of 
the API - not only is it used to store the items added to the CCombo but it is 
vital to size calculations, maintianing selection etc.
Comment 3 Veronika Irvine CLA 2003-11-03 10:08:59 EST 
CTabFolder now creates the tooltip shell only when showing tooltip.
Fixed in branch eclipse_look_3_0.
CCombo will continue to require the shell for its list.