Bug 95844 [Contributions] Unsafe setRedraw() in ToolbarManager.update
This code should protect itself by wrapping the code in a try{} finally{} to ensure the setRedraw(true) happens when it is done.
bundles/org.eclipse.jface/src/org/eclipse/jface/action/ToolBarManager.java
* @return the tool bar control, or <code>null</code> if none (before
	 *         creating or after disposal)
	 */
	public ToolBar getControl() {
		return toolBar;
	}
	/**
	 * Re-lays out the tool bar.
	 * <p>
	 * The default implementation of this framework method re-lays out the
	 * parent when the number of items crosses the zero threshold. Subclasses
	 * should override this method to implement their own re-layout strategy
	 * 
	 * @param layoutBar
	 *            the tool bar control
	 * @param oldCount
	 *            the old number of items
	 * @param newCount
	 *            the new number of items
	 */
	protected void relayout(ToolBar layoutBar, int oldCount, int newCount) {
		if ((oldCount == 0) != (newCount == 0))
			layoutBar.getParent().layout();
	}
	/**
	 * Returns whether the tool bar control is created and not disposed.
	 * 
	 * @return <code>true</code> if the control is created and not disposed,
	 *         <code>false</code> otherwise
	 */