Bug 246243 - Convert debugger view contributions to command framework.
Summary: Convert debugger view contributions to command framework.
Status: NEW
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 3.4   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: JDT-Debug-Inbox CLA
QA Contact:
URL:
Whiteboard: stalebug
Keywords:
: 258984 (view as bug list)
Depends on: 246162 255422
Blocks: 390869
  Show dependency tree
 
Reported: 2008-09-04 12:15 EDT by Pawel Piech CLA
Modified: 2022-11-07 04:45 EST (History)
2 users (show)

See Also:


Attachments
Path with fix. (78.75 KB, patch)
2008-09-04 12:15 EDT, Pawel Piech CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pawel Piech CLA 2008-09-04 12:15:02 EDT
Created attachment 111683 [details]
Path with fix.

As part of investigating how to use the command framework to cleanup the clutter in debugger views I converted the JDT contributions to Debug, Variables, Expressions, and Breakpoints views to the command framework.  The main functional difference is that when using commands, these menu and toolbar items can be hidden by turning off the "Java Debug" action set.  I plan to make a similar contribution to CDT so that users of third-party debuggers would no longer be stuck with a bunch of meaningless icons in these views.

On the implementation side, I had to create a ViewFilterManager which is managed by the plugin activator rather than by the actions as it is now.  This is because the action handlers are not associated with any particular view.  Although in retrospect I could probably still have added the filter registration to the handler in the IElementUpdater.updateElement() method, it really seems like a hack, where as the filter manager is more appropriately managed.

Also, there is still some odd behavior if the user tries to use these commands before the JDI debug plugin is activated, where the checked state of the actions is not updated.  This behavior was actually present when using actions as well, but with commands there is the option of making these items completely invisible, or disabled, until the plugin is activated.
Comment 1 Pawel Piech CLA 2008-09-04 12:16:25 EDT
BTW, I had to use an internal interface to implement some of these commands, but this should be addressed in 3.5.
Comment 2 Pawel Piech CLA 2008-10-27 23:54:17 EDT
I just came accross an older bug 176549, which was marked as "won't fix" because it relied on the debugging activity, so that the actions were available only when the JDT debugger was active.  My patch uses the Java Debug action set instead, so  it can be completely controlled by the user.
Comment 3 Pawel Piech CLA 2008-12-16 13:28:06 EST
*** Bug 258984 has been marked as a duplicate of this bug. ***
Comment 4 Pawel Piech CLA 2008-12-16 13:29:58 EST
A new concern for this bug may be stability issues found with toolbar UI when
using the command framework.
Comment 5 Eclipse Genie CLA 2020-09-12 10:10:04 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.
Comment 6 Eclipse Genie CLA 2022-11-07 04:45:09 EST
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.