Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 342526

Summary: TextSizeProbeStore#probes violates the application scope
Product: [RT] RAP Reporter: Rüdiger Herrmann <ruediger.herrmann>
Component: RWTAssignee: Project Inbox <rap-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: unspecified   
Target Milestone: 1.4 M7   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on:    
Bug Blocks: 337787, 341763    
Attachments:
Description Flags
Fix none

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