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

Bug 330508

Summary: EPartService's bringToTop(MPart) can cause part activation if placeholders are used
Product: [Eclipse Project] e4 Reporter: Remy Suen <remy.suen>
Component: UIAssignee: Remy Suen <remy.suen>
Status: VERIFIED FIXED QA Contact: Remy Suen <remy.suen>
Severity: major    
Priority: P3    
Version: 1.0   
Target Milestone: 4.1 M4   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
EPartService patch v1
none
EPartService patch v2 none

Description Remy Suen CLA 2010-11-17 14:23:40 EST 
1. Activate the editor with a simple Java application in it.
2. Use 'F11' to start debugging it.
3. The 'Debug' view becomes the active part even though the focus is still on the editor. Keybindings like Ctrl+Left and Ctrl+Z will not work.
Comment 1 Remy Suen CLA 2010-11-17 14:28:28 EST 
Caused by the fix for bug 328078. Though the code was bad to begin with. It's just that the fix made it more prevalent.
Comment 2 Remy Suen CLA 2010-11-17 14:39:57 EST 
Created attachment 183334 [details]
EPartService patch v1
Comment 3 Remy Suen CLA 2010-11-17 14:40:34 EST 
(In reply to comment #2)
> Created an attachment (id=183334) [details]
> EPartService patch v1

Fix released to CVS HEAD.
Comment 4 Remy Suen CLA 2010-11-23 14:13:35 EST 
Created attachment 183701 [details]
EPartService patch v2

Found another problem that stemmed from trying to analyze placeholders instead of parts.
Comment 5 Remy Suen CLA 2010-11-23 14:14:53 EST 
(In reply to comment #4)
> Created an attachment (id=183701) [details]
> EPartService patch v2

Fix released to HEAD.
Comment 6 Remy Suen CLA 2010-12-07 14:06:40 EST 
Verified with I20101207-0250 on Windows XP.