Bug 205923 [Viewers] Javadoc for setAllChecked() does not mention that only visible items are checked
setAllChecked which was added (by me) in 3.2 does not mention that all chidren must be visible for it to be checked (see setSubtreeChecked). We will have to leave it as it is API and does not expand the tree but I think we should deprecate it as setSubtreeChecked is really the only useful one. Boris what is your opinion?
bundles/org.eclipse.jface/src/org/eclipse/jface/viewers/CheckboxTreeViewer.java
}
        return false;
    }
    /**
     * Sets to the given value the checked state for all elements in this viewer.
     * Does not fire events to check state listeners.
     *
     * @param state <code>true</code> if the element should be checked,
     *  and <code>false</code> if it should be unchecked
     *  
     *  @since 3.2
     */
	public void setAllChecked(boolean state) {
		setAllChecked(state,  getTree().getItems());
		
	}
	/**
	 * Set the checked state of items and their children to state.
	 * @param state
	 * @param items
	 */
	private void setAllChecked(boolean state, TreeItem[] items) {
		for (int i = 0; i < items.length; i++) {
			items[i].setChecked(state);			
			TreeItem[] children = items[i].getItems();
			setAllChecked(state, children);
		}
	}
}