Bug 195735 - Widget disposal causes NullPointerException
Summary: Widget disposal causes NullPointerException
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.0   Edit
Hardware: PC All
: P1 normal (vote)
Target Milestone: ---   Edit
Assignee: Benjamin Muskalla CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-07-07 06:00 EDT by Rüdiger Herrmann CLA
Modified: 2007-07-12 06:09 EDT (History)
0 users

See Also:

Attachments
Stack trace (2.96 KB, text/plain)
2007-07-07 06:01 EDT, Rüdiger Herrmann CLA 		no flags Details
example snippet (1.85 KB, text/plain)
2007-07-11 19:00 EDT, Benjamin Muskalla CLA 		no flags Details
View All 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 Rüdiger Herrmann CLA 2007-07-07 06:00:04 EDT 
CVS HEAD

Steps to reproduce:
1. launch workbench demo with -console program argument
2. click on the windows' close button or choose File -> Exit from the menu
=> see the NullPointerException in the console output
Comment 1 Rüdiger Herrmann CLA 2007-07-07 06:01:13 EDT 
Created attachment 73242 [details]
Stack trace
Comment 2 Benjamin Muskalla CLA 2007-07-11 12:25:04 EDT 
This problem occurs also in other dispose situations. It seems that "something" can cause the widget field in the lca is null but the currentWidgetRef has a value -> ensureWidgetRef() tries to set the new widget ref and fails to due to the null of the widget field. Checking widget against null in JSWriter:506 (maybe outdated) solves this problem for me. But I'm not sure if it's the cause for other problems (have much trouble at the moment on other sides - not yet sure if it's related). 

I think to should be definitely fixed before the M5 milestone. Will try to provide a snippet to reproduce this situation. Changes priority to P1
Comment 3 Benjamin Muskalla CLA 2007-07-11 19:00:16 EDT 
Created attachment 73612 [details]
example snippet

The problem is the order in which the widgets are disposed.
Comment 4 Benjamin Muskalla CLA 2007-07-12 06:09:48 EDT 
Fixed in CVS.