Bug 304501 WBWRenderer has dead code
A layout is set on line 292... wbwShell.setLayout(new FillLayout()); ...then a layout is set again on line 305... if (element instanceof MWindow) { TrimmedPartLayout tl = new TrimmedPartLayout(wbwShell); wbwShell.setLayout(tl); } else { wbwShell.setLayout(new FillLayout()); } ...but the instanceof check was already done in the beginning. The TPL should just have been set on line 292 to begin with.
bundles/org.eclipse.e4.ui.workbench.renderers.swt/src/org/eclipse/e4/workbench/ui/renderers/swt/WBWRenderer.java
private class WindowSizeUpdateJob implements Runnable {
		public List<MWindow> windowsToUpdate = new ArrayList<MWindow>();
		public void run() {
			clearSizeUpdate();
			while (!windowsToUpdate.isEmpty()) {
				MWindow window = windowsToUpdate.remove(0);
				Shell shell = (Shell) window.getWidget();
				if (shell == null || shell.isDisposed())
					continue;
				shell.setBounds(window.getX(), window.getY(),
						window.getWidth(), window.getHeight());
			}
		}
	}
	WindowSizeUpdateJob boundsJob;
	void clearSizeUpdate() {
		boundsJob = null;
	}
	boolean ignoreSizeChanges = false;
	@Inject
	Logger logger;
	@Inject
	private IEventBroker eventBroker;
	private EventHandler shellUpdater;
	private EventHandler visibilityHandler;
	private EventHandler sizeHandler;
	public WBWRenderer() {