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 183721 - AssertionError at org.netbeans.spi.project.support.ant.SequentialPropertyEvaluator.stateChanged
Summary: AssertionError at org.netbeans.spi.project.support.ant.SequentialPropertyEval...
Status: VERIFIED FIXED
Alias: None
Product: javafx
Classification: Unclassified
Component: Unsupported (show other bugs)
Version: 6.x
Hardware: All All
: P1 normal (vote)
Assignee: Adam Sotona
URL:
Keywords: REGRESSION, THREAD
Depends on:
Blocks: 181855
  Show dependency tree
 
Reported: 2010-04-08 15:18 UTC by J Bachorik
Modified: 2010-07-22 12:47 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter: 167779

Attachments
stacktrace (5.51 KB, text/plain)
2010-04-08 15:19 UTC, J Bachorik 		Details
stacktrace (5.44 KB, text/plain)
2010-04-12 10:12 UTC, Alexandr Scherbatiy 		Details
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 J Bachorik 2010-04-08 15:18:57 UTC 
Build: NetBeans IDE Dev (Build 201004070201)
VM: Java HotSpot(TM) 64-Bit Server VM, 14.3-b01, Java(TM) SE Runtime Environment, 1.6.0_17-b04
OS: Linux

Stacktrace: 
java.lang.AssertionError
   at org.netbeans.spi.project.support.ant.SequentialPropertyEvaluator.stateChanged(SequentialPropertyEvaluator.java:160)
   at org.openide.util.WeakListenerImpl$Change.stateChanged(WeakListenerImpl.java:321)
   at org.openide.util.ChangeSupport.fireChange(ChangeSupport.java:130)
   at org.openide.util.ChangeSupport.fireChange(ChangeSupport.java:116)
   at org.netbeans.spi.project.support.ant.FilterPropertyProvider.setDelegate(FilterPropertyProvider.java:91)
   at org.netbeans.modules.javafx.project.JavaFXProject$ConfigPropertyProvider.run(JavaFXProject.java:238)
Comment 1 J Bachorik 2010-04-08 15:19:01 UTC 
Created attachment 96908 [details]
stacktrace
Comment 2 Alexandr Scherbatiy 2010-04-12 10:12:21 UTC 
Created attachment 97060 [details]
stacktrace

NetBeans-JavaFX-Soma:  #217

Stesp to reproduce:

- Create a JavaFX project
- Change the Default profile to the Browser in the drop down list on the toolbar

The exception pops uo
Comment 3 Alexandr Scherbatiy 2010-04-12 10:13:07 UTC 
It is a regression for the NB JavaFX module
Comment 4 Jesse Glick 2010-04-12 15:26:36 UTC 
JavaFXProject$ConfigPropertyProvider.run is changing metadata without a lock on the project mutex. Wrap in:

ProjectManager.mutex().writeAccess(...)

or just do not post to RP (since the calling thread had the lock already).
Comment 5 Jesse Glick 2010-04-12 15:27:34 UTC 
BTW this assertion is unchanged since 6.8 (at least), so if it a regression, it is in FX code.
Comment 6 Adam Sotona 2010-04-12 15:49:28 UTC 
Please remove the assert,
changes are fired in another thread to avoid holding Project.writeMutex when the listeners are notified.

This has been changed to fix issue #181855
Comment 7 Jesse Glick 2010-04-12 16:43:05 UTC 
The fix of bug #181855 was incorrect; the asynchronous part should be in PlatformNode, not ConfigPropertyProvider. So back out 4351e53f3352, probably retaining 538f0d0123c7.
Comment 8 Adam Sotona 2010-04-13 08:09:19 UTC 
OK, backed out 4351e53f3352 here: http://hg.netbeans.org/javafx/rev/4993c3cafc44
Comment 9 Alexandr Scherbatiy 2010-04-13 10:07:29 UTC 
verified in build NetBeans-JavaFX-Soma: #219