Bug 337002 EPartService leaks previously active part
A reference is kept to the previously active part. This is unnecessary. We only need it to fire events. After the deactivation events have been fired, we should no longer need to keep it lying around.
tests/org.eclipse.e4.ui.tests/src/org/eclipse/e4/ui/tests/application/EPartServiceTest.java
testSaveAll_NoHandlers(confirm, beforeDirty,
				afterDirty(beforeDirty, throwException),
				isSuccessful(beforeDirty, throwException),
				saveCalled(beforeDirty, throwException), throwException);
	}
	public void testSaveAll_NoHandlers_T_TT_TT() {
		testSaveAll_NoHandlers(true, new boolean[] { true, true },
				new boolean[] { true, true });
	}
	public void testSaveAll_NoHandlers_T_TT_TF() {
		testSaveAll_NoHandlers(true, new boolean[] { true, true },
				new boolean[] { true, false });
	}
	public void testSaveAll_NoHandlers_T_TT_FT() {
		testSaveAll_NoHandlers(true, new boolean[] { true, true },
				new boolean[] { false, true });
	}
	public void testSaveAll_NoHandlers_T_TT_FF() {
		testSaveAll_NoHandlers(true, new boolean[] { true, true },
				new boolean[] { false, false });
	}
	public void testSaveAll_NoHandlers_T_TF_TT() {
		testSaveAll_NoHandlers(true, new boolean[] { true, false },
				new boolean[] { true, true });
	}
	public void testSaveAll_NoHandlers_T_TF_TF() {
		testSaveAll_NoHandlers(true, new boolean[] { true, false },
				new boolean[] { true, false });
	}
	public void testSaveAll_NoHandlers_T_TF_FT() {