This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 167708 - Wait cursor shows while selecting file in projects view
Summary: Wait cursor shows while selecting file in projects view
Status: CLOSED FIXED
Alias: None
Product: java
Classification: Unclassified
Component: Navigation (show other bugs)
Version: 6.x
Hardware: All All
: P3 blocker (vote)
Assignee: David Strupl
URL:
Keywords: PERFORMANCE
: 70708 (view as bug list)
Depends on: 169353
Blocks:
  Show dependency tree
 
Reported: 2009-06-26 16:32 UTC by Marian Mirilovic
Modified: 2010-04-22 10:40 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
Profiler snapshot (11.58 KB, application/octet-stream)
2009-06-26 16:33 UTC, Marian Mirilovic
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Marian Mirilovic 2009-06-26 16:32:37 UTC
NB 6.7 FCS , JDK 6u14 64b, Ubuntu 9.04 64b

Steps to reproduce:
- run IDE with new userdir
- create sample Java project (e.g. Mars Rover Viewer)
- wait until project is opened (keep Navigator opened)
- select another Java File
-> wait cursor is shown for ~500ms and disappears

I do not think we need to show cursor for this operation, it doesn't block user while working in Projects view (invoke
popup/open file/ ....) ... BTW: Navigator itself displays "Please Wait..." and it's enough -> from my point of view.

see attached snapshot
Comment 1 Marian Mirilovic 2009-06-26 16:33:25 UTC
Created attachment 84060 [details]
Profiler snapshot
Comment 2 David Strupl 2009-07-28 15:30:25 UTC
This is the problematic stack trace where the busy cursor gets there:

java.lang.Exception
        at java.awt.Component.setCursor(Component.java:2568)
        at org.openide.explorer.view.TreeView.doShowWaitCursor(TreeView.java:880)
        at org.openide.explorer.view.TreeView.showWaitCursor(TreeView.java:872)
        at org.openide.explorer.view.TreeView.prepareWaitCursor(TreeView.java:908)
        at org.openide.explorer.view.TreeView.access$800(TreeView.java:143)
        at org.openide.explorer.view.TreeView$TreePropertyListener.treeWillExpand(TreeView.java:1419)
        at javax.swing.JTree.fireTreeWillExpand(JTree.java:2302)
        at javax.swing.JTree.setExpandedState(JTree.java:3000)
        at javax.swing.JTree.expandPath(JTree.java:1742)
        at javax.swing.JTree.expandRow(JTree.java:1757)
        at org.openide.explorer.view.TreeView.expandAll(TreeView.java:594)
        at org.netbeans.modules.java.navigation.ClassMemberPanelUI$MyBeanTreeView.expandAll(ClassMemberPanelUI.java:477)
        at org.netbeans.modules.java.navigation.ClassMemberPanelUI$3.run(ClassMemberPanelUI.java:181)
        at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:209)
        at java.awt.EventQueue.dispatchEvent(EventQueue.java:461)
        at org.netbeans.core.TimableEventQueue.dispatchEvent(TimableEventQueue.java:117)
        at java.awt.EventDispatchThread.pumpOneEventForHierarchy(EventDispatchThread.java:242)
        at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:163)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:157)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:149)
        at java.awt.EventDispatchThread.run(EventDispatchThread.java:110)
Comment 3 David Strupl 2009-07-28 15:44:50 UTC
Marking this as P3.
Comment 4 Marian Mirilovic 2009-07-29 13:09:20 UTC
David, P2/3 doesn't matter ... but please fix this for 6.8
Comment 5 David Strupl 2009-09-21 09:43:27 UTC
I cannot fix this in 6.8 as it depends on 169353. Changing to "future" and removing the keyword.
Comment 6 Tomas Pavek 2009-09-21 14:16:47 UTC
Is the problem really in expanding (as stated in issue 169353)? The problem reported by Marian is just about 
selecting. I'd say wait cursor for expanding is ok, but not for selecting.
Comment 7 David Strupl 2009-09-21 15:07:57 UTC
It is about expanding. When you select a file in projects the *other* explorer bellow (Navigator) changes its root node
and tries to expand the root node to show you the methods, fields etc. BTW that's why I have pasted the stack trace to
the comment which shows it clearly how it gets there (via TreeView.expandAll).
Comment 8 David Strupl 2009-10-20 13:20:27 UTC
http://hg.netbeans.org/jet-main/rev/5ab7a270b5a0
Comment 9 David Strupl 2009-10-20 13:31:49 UTC
Applied to CSL as well: http://hg.netbeans.org/jet-main/rev/43034a05eb5c
Comment 10 Marian Mirilovic 2009-10-22 15:00:09 UTC
*** Issue 70708 has been marked as a duplicate of this issue. ***
Comment 11 Marian Mirilovic 2010-04-22 10:40:41 UTC
v/c