_activePageState in StatemManagerImpl should be on the session level
--------------------------------------------------------------------
Key: TRINIDAD-1289
URL: https://issues.apache.org/jira/browse/TRINIDAD-1289
Project: MyFaces Trinidad
Issue Type: Bug
Affects Versions: 1.2.9-core
Reporter: Stevan Malesevic
StateManagerImpl has a code like this:
synchronized(this)
{
if (_activePageState != null)
_activePageState.clearViewRootState();
_activePageState = pageState;
}
There are two problems here:
1. Even when the last user session on the system expires the memory under
PageState, which holds onto UIViewRoot, will not be gc-ed since Application
holds onto StateManagerImpl which holds onto PageState
2. _activePageState is member of StateManagerImpl, which means different user
sessions will call clearViewRootState on each other
Basically, _activePageState should be kept on Session so that both of these
issues are taken care of
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
