Bug 353342 [Compatibility] IWorkingSetManager's

testRecentWorkingSetsLength is failing

Since the working set manager now supports the setting of the
number of recent working sets to remember, the constant that was
originally defined is no longer required. The manager should
instead size itself based on the defined number of recent
working sets to remember.

1 files changed, 1 insertions, 7 deletions

diff --git a/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/AbstractWorkingSetManager.java b/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/AbstractWorkingSetManager.java
index 92584ee13a1..fd3d8f2fa72 100644
--- a/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/AbstractWorkingSetManager.java
+++ b/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/AbstractWorkingSetManager.java
@@ -100,10 +100,6 @@ public abstract class AbstractWorkingSetManager extends EventManager implements
 		}
 	});
     
-    /**
-     * Size of the list of most recently used working sets.
-     */
-    private static final int MRU_SIZE = 5;
     private List recentWorkingSets = new ArrayList();
 
     private BundleContext bundleContext;
@@ -323,9 +319,7 @@ public abstract class AbstractWorkingSetManager extends EventManager implements
 			}
         recentWorkingSets.remove(workingSet);
         recentWorkingSets.add(0, workingSet);
-        if (recentWorkingSets.size() > MRU_SIZE) {
-            recentWorkingSets.remove(MRU_SIZE);
-        }
+		sizeRecentWorkingSets();
     }
 
     //---- equals and hash code -----------------------------------------------