Bug 178888 [Viewers] JavaDoc errors in GridDataFactory/GridLayoutFactory
There are several javadoc errors in GridDataFactory/GridLayoutFactory. The GridDataFactory class comment contains an "IMPORTANT" comment which was recently rewritten to add javadoc tags... however the meaning was changed subtly and now it warns the user against doing something that is perfectly valid. The GridLayoutFactory Javadoc still references the old name for the extendedMargins method.
bundles/org.eclipse.jface/src/org/eclipse/jface/layout/GridLayoutFactory.java
import org.eclipse.swt.graphics.Rectangle;
import org.eclipse.swt.layout.GridLayout;
import org.eclipse.swt.widgets.Composite;
/**
 * GridLayoutFactory creates and initializes grid layouts. There are two ways to use GridLayoutFactory.
 * Normally, it is used as a shorthand for writing "new GridLayout()" and initializing a bunch 
 * of fields. In this case the main benefit is a more concise syntax and the ability to create more
 * than one identical GridLayout from the same factory. Changing a property of the factory will affect
 * future layouts created by the factory, but has no effect on layouts that have already been created.
 * 
 * <p>
 * GridLayoutFactory can also generate grid data for all the controls in a layout. This is done with
 * the generateLayout method. To use this feature:
 * </p>
 * 
 * <ol>
 * <li>Create the composite</li>
 * <li>Create all the controls in the composite</li>
 * <li>Call generateLayout</li>
 * </ol>
 * 
 * <p>
 * The order here is important. generateLayout must be called after all the child controls have
 * been created. generateLayout will not change any layout data that has already been attached
 * to a child control and it will not recurse into nested composites. 
 * </p>
 *
 * @since 3.2
 */
public final class GridLayoutFactory {