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 212895 - Slow search for node at caret position in Navigator tree
Summary: Slow search for node at caret position in Navigator tree
Status: RESOLVED WONTFIX
Alias: None
Product: editor
Classification: Unclassified
Component: CSL (API & infrastructure) (show other bugs)
Version: 7.2
Hardware: All All
: P3 normal (vote)
Assignee: Svata Dedic
URL:
Keywords: PERFORMANCE
: 212996 214058 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-05-23 12:50 UTC by KhArtNJava
Modified: 2016-07-07 07:30 UTC (History)
40 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter: 188254


Attachments
nps snapshot (50.63 KB, application/nps)
2012-05-23 12:50 UTC, KhArtNJava
Details

Note You need to log in before you can comment on or make changes to this bug.
Description KhArtNJava 2012-05-23 12:50:14 UTC
This bug was originally marked as duplicate of bug 211927, that is already resolved. This bug is still valid, so this seems to be another bug, but it might be related.

Build: NetBeans IDE Dev (Build 201205200400)
VM: Java HotSpot(TM) Client VM, 22.1-b02, Java(TM) SE Runtime Environment, 1.7.0_03-b05
OS: Windows 7
Maximum slowness yet reported was 6958 ms, average is 6958
Comment 1 KhArtNJava 2012-05-23 12:50:22 UTC
Created attachment 119773 [details]
nps snapshot
Comment 2 Marek Fukala 2012-05-23 13:13:53 UTC
setting caret in the editor triggers ElementNode.getNodeForOffset() which tries to find the leaf-most element in the tree hierarchy which requires recursive calls to HtmlStructureItems.getNestedItems() (which does PM.runUserTask(...) )
Comment 3 Marek Fukala 2012-05-25 10:45:58 UTC
*** Bug 212996 has been marked as a duplicate of this bug. ***
Comment 4 Svata Dedic 2012-05-25 10:57:06 UTC
In report 582233, the AWT is blocked by mere parent Node expansion. Children of the HTML node must be parsed, but parsing loop is evaluating HTML hints in the report.
.
It would be probably better to create child nodes in a RP, and display a temporary 'Please wait' pseudo-node, iff the Item does not return nodes within specified time (100ms or so)
Comment 5 Marek Fukala 2012-05-25 12:28:51 UTC
my comment #2 is related to the report 581730
Comment 6 Svata Dedic 2012-06-11 12:49:16 UTC
Changeset: 02c8dd0a32eb
Author:    Svata Dedic <sdedic@netbeans.org>
Date:      2012-06-11 14:49
Message:   Issue #212895 - LowPerformance took 6958 ms.: fixed
Calls to getNestedItems() made asynchronous, Navigator paints WAIT node if nested items cannot be fetched in 100ms.
Comment 7 Milutin Kristofic 2012-06-14 11:21:50 UTC
*** Bug 214058 has been marked as a duplicate of this bug. ***
Comment 8 Quality Engineering 2012-06-15 06:14:15 UTC
Integrated into 'main-golden', will be available in build *201206150001* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/02c8dd0a32eb
User: Svata Dedic <sdedic@netbeans.org>
Log: Issue #212895 - LowPerformance took 6958 ms.: fixed
Calls to getNestedItems() made asynchronous, Navigator paints WAIT node if nested items cannot be fetched in 100ms.
Comment 9 Svata Dedic 2012-06-19 11:23:55 UTC
The change was reverted in trunk (pending propagation into 7.2 branch under HR rules), see issue #214421. Related issues introduced by the change:
* issue #214178
* issue #214316
Comment 10 Quality Engineering 2012-06-22 04:20:39 UTC
Integrated into 'releases', will be available in build *201206212341* or newer. Wait for official and publicly available build.
Changeset: http://hg.netbeans.org/releases/rev/340b333e2f51
User: Svata Dedic <sdedic@netbeans.org>
Log: Issue #214421 - Withdraw poorly tested fix for #212895: fixed
Reverted improper fix for issue #212895
Comment 11 Quality Engineering 2012-06-22 04:47:42 UTC
Integrated into 'main-golden', will be available in build *201206220002* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/b311735c9afc
User: Svata Dedic <sdedic@netbeans.org>
Log: Issue #214421 - Withdraw poorly tested fix for #212895: fixed
Reverted improper fix for issue #212895
Comment 12 Vladimir Riha 2012-08-10 09:55:54 UTC
My last report 605288 is related to running html in Chrome with IDE integration turned on. Should I separate it?
Comment 13 Martin Balin 2016-07-07 07:30:52 UTC
This old bug may not be relevant anymore. If you can still reproduce it in 8.2 development builds please reopen this issue.

Thanks for your cooperation,
NetBeans IDE 8.2 Release Boss