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/GridDataFactory.java
/**
     * Returns a GridDataFactory initialized with heuristicly generated defaults for the given control.
     * To be precise, this method picks the default values that GridLayoutFactory.generateLayout
     * would have assigned to the control. Does not attach GridData to the control. Callers must
     * additionally call applyTo(theControl) if they wish to use the generated values.
     * 
     * <p>
     * This method is intended for situations where generateLayout is generating layout data
     * for a particular control that is not quite right for the desired layout.
     * This allows callers to start with the generated values and tweak one or two settings
     * before applying the GridData to the control.
     * </p> 
     * 
     * @see GridLayoutFactory#generateLayout(org.eclipse.swt.widgets.Composite)
     * @param theControl 
     * @return a GridLayoutFactory initialized with defaults that GridLayoutFactory would have 
     * @since 3.3
     */
    public static GridDataFactory defaultsFor(Control theControl) {
    	return LayoutGenerator.defaultsFor(theControl);
    }
    
    /**
     * Generates layout data to the given control, given the number of cells
     * spanned by the control. Attaches a GridData to the control. This method 
     * allows generated layout data to be used with controls that span multiple cells. 
     * <p>
     * The generated layout data is the same as what would be generated by 
     * GridLayoutFactory.generateLayout, except that the span is configurable
     * </p>