Bug 163317 [Viewers] Take over SWT programming pattern to JFace Viewers
To open ColumnViewer for reusablility we should not make it final making the constructor protected forces people to subclass it and not directly creating instances from it.
bundles/org.eclipse.ui.ide/src/org/eclipse/ui/views/markers/internal/TableView.java
* 
							 * @see java.lang.Runnable#run()
							 */
							public void run() {
								resortTable(column, field,
										new NullProgressMonitor());
							}
						});
					else
						getProgressService().busyCursorWhile(
								new IRunnableWithProgress() {
									/*
									 * (non-Javadoc)
									 * 
									 * @see org.eclipse.jface.operation.IRunnableWithProgress#run(org.eclipse.core.runtime.IProgressMonitor)
									 */
									public void run(IProgressMonitor monitor) {
										resortTable(column, field, monitor);
									}
								});
				} catch (InvocationTargetException e1) {
					IDEWorkbenchPlugin.getDefault().getLog().log(
							Util.errorStatus(e1));
				} catch (InterruptedException e1) {
					return;
				}
			}
			/**
			 * Resort the table based on field.
			 *