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.
Performance test reports there is a NetBeans startup regression which is caused by the loading of the following class: org.netbeans.modules.javahelp.JavaHelpQuickSearchProviderImpl Please don't load this class during NetBeans startup. Comment from jtulach: "No initialization of providers before search is really done (and even after that only some shall be initialized)."
Created attachment 66693 [details] Stacktrace
NetBeans startup regression is caused by the loading of the following classes: org.netbeans.modules.javahelp.JavaHelpQuickSearchProviderImpl org.netbeans.modules.jumpto.quicksearch.JavaTypeSearchProvider org.netbeans.modules.options.QuickSearchProvider org.netbeans.modules.options.keymap.ActionsSearchProvider org.netbeans.modules.quicksearch.ProviderModel org.netbeans.modules.quicksearch.ProviderModel$Category org.netbeans.modules.quicksearch.ProviderRegistry org.netbeans.modules.quicksearch.QuickSearchComboBar$DynamicWidthTA org.netbeans.modules.quicksearch.recent.RecentProvider Please don't load these classes during NetBeans startup.
Created attachment 66696 [details] Stacktraces
And also the following class: org.netbeans.spi.quicksearch.SearchProvider
Created attachment 66698 [details] Stacktrace
One more class to this list: org.netbeans.modules.quicksearch.CommandEvaluator
Created attachment 66714 [details] Stacktrace
Hm, I18N requests and perf requests are in clash here. I18N team request leads to code that has to load display names of all categories during first display of qsearch field (=during startup). OK, I rewrote code to load Quick Search providers lazily when they are needed (when user types into search box). However, UI of Quick Search is visible in main window right from start, so UI classes of Quick Search must be loaded and used during startup, no other way here. Fixed: http://hg.netbeans.org/core-main/cfccf60a1305 http://hg.netbeans.org/main/cfccf60a1305
> However, UI of Quick Search is visible in main window right from start, so UI classes > of Quick Search must be loaded and used during startup, no other way here. No dispute. Just note that the reported classes appeared newly - there was some change during last two weeks that caused that; they were not loaded before. Which ones do you think can't be avoided?
Integrated into 'main-golden', available in build *200808090201* on http://bits.netbeans.org/dev/nightly/ Changeset: http://hg.netbeans.org/main/rev/cfccf60a1305 User: Dafe Simonek <dsimonek@netbeans.org> Log: #143052: load provider classes more lazily, so that they are instantiated right before quick search is performed by user, not sonner
After my fix, only org.netbeans.modules.quicksearch.ProviderModel, org.netbeans.modules.quicksearch.ProviderModel$Category, and CommandEvaluator are needed - they are used to obtain and store display names of categories which are used to compute preferred width of the text box (fix of 140789)
With 080811 Dev build test result I see that the following classes are gone: org.netbeans.modules.javahelp.JavaHelpQuickSearchProviderImpl org.netbeans.modules.jumpto.quicksearch.JavaTypeSearchProvider org.netbeans.modules.options.QuickSearchProvider org.netbeans.modules.options.keymap.ActionsSearchProvider org.netbeans.modules.quicksearch.ProviderRegistry org.netbeans.modules.quicksearch.recent.RecentProvider org.netbeans.spi.quicksearch.SearchProvider while the following classes are still being loaded: org.netbeans.modules.quicksearch.CommandEvaluator org.netbeans.modules.quicksearch.ProviderModel org.netbeans.modules.quicksearch.ProviderModel$Category org.netbeans.modules.quicksearch.QuickSearchComboBar$DynamicWidthTA dsimonek, what do you think about the last one? jtulach, do you agree to add these classes to whitelist?
org.netbeans.modules.quicksearch.QuickSearchComboBar$DynamicWidthTA is text box itself, must be here to return dynamically calculated preferred size
I think the last four classes are ok.