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
} else {
					Arrays.fill(response, Save.NO);
					for (int i = 0; i < elements.length; i++) {
						response[parts.indexOf(elements[i])] = Save.YES;
					}
				}
				return response;
			}
		});
	}
	@PreDestroy
	public void contextDisposed() {
		eventBroker.unsubscribe(shellUpdater);
		eventBroker.unsubscribe(visibilityHandler);
		eventBroker.unsubscribe(sizeHandler);
	}
	public Object createWidget(MUIElement element, Object parent) {
		final Widget newWidget;
		if (!(element instanceof MWindow)
				|| (parent != null && !(parent instanceof Shell)))
			return null;
		MWindow wbwModel = (MWindow) element;
		Shell parentShell = (Shell) parent;
		IEclipseContext parentContext = getContextForParent(element);
		Shell wbwShell;
		if (parentShell == null) {
			wbwShell = new Shell(Display.getCurrent(), SWT.SHELL_TRIM);
		} else if (wbwModel.getTags().contains(DragHost.DragHostId)) {
			wbwShell = new Shell(parentShell, SWT.BORDER);
			wbwShell.setAlpha(110);