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
data.heightHint = 250;
			data.widthHint = 300;
			tableViewer.getControl().setLayoutData(data);
			tableViewer.setLabelProvider(new LabelProvider() {
				@Override
				public String getText(Object element) {
					return ((MPart) element).getLabel();
				}
			});
			tableViewer.setContentProvider(ArrayContentProvider.getInstance());
			tableViewer.setInput(collection);
			tableViewer.setAllChecked(true);
			return parent;
		}
		@Override
		public void create() {
			super.create();
			applyDialogStyles(getShell());
		}
		@Override
		protected void okPressed() {
			checkedElements = tableViewer.getCheckedElements();
			super.okPressed();
		}
		public Object[] getCheckedElements() {
			return checkedElements;
		}
	}
}