Bug 361652 - [JFace] FieldDecorationRegistry#getDefault should have session scope
Summary: [JFace] FieldDecorationRegistry#getDefault should have session scope
Status: ASSIGNED
Alias: None
Product: RAP
Classification: RT
Component: JFace (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-21 09:56 EDT by Rüdiger Herrmann CLA
Modified: 2012-03-21 05:21 EDT (History)
0 users

See Also:

Attachments
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 2011-10-21 09:56:07 EDT 
The default FieldDecorationRegistry currently has application scope that can be set and obtained with get/setDefault(). It is stored in a static field. This will likely cause memory leaks as the managed FieldDecorationRegistry have session scope. 
Furthermore, access is unsynhronized and id's from different sessions may override each other inadvertently.

The default FieldDecorationRegistry should be stored in session scope.