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 235987

Summary:	org.netbeans.modules.cnd.makeproject.api.actions.AddExistingItemAction.performAction: LowPerformance took 31097 ms.
Product:	cnd	Reporter:	Exceptions Reporter <exceptions_reporter>
Component:	Project	Assignee:	Maria Tishkova <mromashova>
Status:	VERIFIED FIXED
Severity:	normal	Keywords:	PERFORMANCE
Priority:	P3
Version:	7.3
Hardware:	All
OS:	All
Issue Type:	DEFECT	Exception Reporter:	200586
Attachments:	nps snapshot

Description Exceptions Reporter 2013-09-15 11:15:14 UTC

Build: NetBeans IDE 7.3.1 (Build 201306052037)
VM: Java HotSpot(TM) 64-Bit Server VM, 23.25-b01, Java(TM) SE Runtime Environment, 1.7.0_25-b17
OS: Windows 7

User Comments:
GUEST: 1. Adding existing item to project
2. Click desktop in the file dialog

GUEST: Low Performance while i was opening a new file



Maximum slowness yet reported was 31097 ms, average is 26413

Comment 1 Exceptions Reporter 2013-09-15 11:15:16 UTC

Created attachment 140095 [details]
nps snapshot

Comment 2 Vladimir Voskresensky 2013-12-24 16:23:36 UTC

Maria, have a look. 
If you put tracing breakpoint and text containing delay, like
{= Thread.sleep(2000);}

 in BasicFileChooserUI.BasicFileView.getIcon(f)
at line 
icon = directoryIcon
you will see hang up of UI even on Unix

Comment 3 Maria Tishkova 2013-12-29 20:16:12 UTC

fixed in http://hg.netbeans.org/cnd-main/rev/319a080c1f09

Comment 4 Quality Engineering 2013-12-30 12:04:32 UTC

Integrated into 'main-silver', will be available in build *201312300942* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)

Changeset: http://hg.netbeans.org/main-silver/rev/319a080c1f09
User: Maria Dalmatova <mromashova@netbeans.org>
Log: fixed bz#235987  org.netbeans.modules.cnd.makeproject.api.actions.AddExistingItemAction.performAction: LowPerformance took 31097 ms.
run loading of icon in separate thread.
Already fixed a small issue with invoking re-scan directory of chooser ui from non-ui UpdateWorker thread

Comment 5 soldatov 2014-03-21 17:18:46 UTC

I don't see any regressions in latest test report [NetBeans IDE 7.4 (Build 20140320-3f80b6998498)]

Comment 6 Quality Engineering 2014-03-26 17:06:16 UTC

Integrated into 'releases/release74', will be available in build *201403261440* or newer. Wait for official and publicly available build.

Changeset: http://hg.netbeans.org/releases/rev/e825cfd49cc0
User: Maria Dalmatova <mromashova@netbeans.org>
Log: fixed bz#235987  org.netbeans.modules.cnd.makeproject.api.actions.AddExistingItemAction.performAction: LowPerformance took 31097 ms.
run loading of icon in separate thread.
Already fixed a small issue with invoking re-scan directory of chooser ui from non-ui UpdateWorker thread
(transplanted from 319a080c1f0972b987bb2481bf55c35dcd9cd5f4)