Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 342526 - TextSizeProbeStore#probes violates the application scope
Summary: TextSizeProbeStore#probes violates the application scope
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.4 M7   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 337787 341763
  Show dependency tree
 
Reported: 2011-04-12 05:15 EDT by Rüdiger Herrmann CLA
Modified: 2011-04-13 13:49 EDT (History)
0 users

See Also:

Attachments
Fix (33.65 KB, patch)
2011-04-13 13:48 EDT, Rüdiger Herrmann CLA 		no flags Details | Diff
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 2011-04-12 05:15:02 EDT 
The TextSizeProbeStore has a static field 'probes' which exceeds the application scope that the TextSizeProbeStore should live in.
This bug also forces tests to use TextSizeProbeStore#reset() on tear down. This method is for tests only. Otherwise it could be deleted.
Comment 1 Rüdiger Herrmann CLA 2011-04-13 13:48:27 EDT 
Created attachment 193179 [details]
Fix

This patch moves all methods that handle probe results to the TextSizeProbeResults class.
TextSizeProbeStore no longer has static methods but has become an instance in the ApplicationContext
Comment 2 Rüdiger Herrmann CLA 2011-04-13 13:49:17 EDT 
Applied patch to CVS HEAD