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

Bug 329950

Summary:

'Close Others' and 'Close All' may cause bundle activation

Product:

[Eclipse Project] e4

Reporter:

Remy Suen <remy.suen>

Component:

Assignee:

Remy Suen <remy.suen>

Status:

RESOLVED FIXED

QA Contact:

Remy Suen <remy.suen>

Severity:

major

Priority:

Version:

1.0

Target Milestone:

4.1 M4

Hardware:

All

OS:

All

Whiteboard:

Attachments:

Description	Flags
StackRenderer patch v1	none

Description Remy Suen

2010-11-10 16:43:06 EST

Comment 1 Remy Suen

2010-11-10 16:44:59 EST

Whoops, hit enter by mistake.

1. Ctrl+3 > Console
2. Ctrl+3 > Plug-ins
3. Activate the 'Declaration' view. It should be the third one out of the five.
4. Shutdown.
5. Restart.
6. Add a breakpoint to CompatibilityPart's constructor.
7. Right-click on the 'Plug-ins' view's tab > Close Others
8. The breakpoint is hit twice, once for the 'Console' view and the second time for the 'Plug-ins' view.

Comment 2 Remy Suen

2010-11-10 19:52:33 EST

Shortened versions:
1. Add a breakpoint to CompatibilityPart's constructor.
2. Right-click on the 'Declaration' view's tab > Close Others
3. The breakpoint is hit twice, once for the 'Javadoc' view and the second time
for the 'Declaration' view. Only the 'Declaration' view should have been instantiated.

1. Add a breakpoint to CompatibilityPart's constructor.
2. Right-click on the 'Declaration' view's tab > Close All
3. The breakpoint is hit twice, once for the 'Javadoc' view and the second time for the 'Declaration' view. Neither of these two views should have been instantiated.

Comment 3 Remy Suen

2010-11-10 20:27:17 EST

Created attachment 182866 [details]
StackRenderer patch v1

Patch to place the selected element as the last element to hide so that we don't inadvertently change our selection (and cause part creation) when hiding the parts.

Comment 4 Remy Suen

2010-11-10 20:28:41 EST

Fix released to CVS HEAD.