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 165159 - go to source breaks after sorting table
Summary: go to source breaks after sorting table
Status: RESOLVED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: Outline&TreeTable (show other bugs)
Version: 6.x
Hardware: All All
: P3 blocker (vote)
Assignee: Martin Entlicher
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-05-14 13:12 UTC by Alexey Vladykin
Modified: 2009-08-01 06:33 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Vladykin 2009-05-14 13:12:15 UTC 
1. Profile any project.
2. Click on indicator to open details.
3. Double-click on functions in details to verify that correct sources are opened.
4. Change sort order of the details table (click on any column header).
=> double-clicking on functions in details opens incorrect source locations.
Comment 1 Alexey Vladykin 2009-05-14 13:13:09 UTC 
Happens on Solaris with DTrace data collector. Haven't checked others.
Comment 2 Alexey Vladykin 2009-05-15 17:22:31 UTC 
Same thing happens if source info prefetching is turned off.
Comment 3 Alexey Vladykin 2009-05-15 17:58:34 UTC 
Reassigning to platform component for evaluation.
It seems that Outline view does not take into account current node permutation when processing double click.
I put breakpoint in OutlineView.PopupAdapter.mouseClicked, and selPath was different from what I double-clicked on.
Comment 4 Martin Entlicher 2009-07-31 14:31:27 UTC 
Reproduced also with debugger views.
Comment 5 Martin Entlicher 2009-07-31 15:49:33 UTC 
Fixed in changeset:   139951:f4f0b7a5e5bf
http://hg.netbeans.org/main/rev/f4f0b7a5e5bf
Comment 6 Quality Engineering 2009-08-01 06:33:12 UTC 
Integrated into 'main-golden', will be available in build *200908010201* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/f4f0b7a5e5bf
User: mentlicher@netbeans.org
Log: #165159 - getClosestPathForLocation() corrected - UI row needs to be converted to the model row.