Fix PrecisionRectangle#getBottom inconsistency

The y value of PrecisionRectangle#getBottom can return a different
y value to PrecisionRectangle#getBottomLeft and
PrecisionRectangle#getBottomRight because the first uses the int values
the later two use precision values. This commit adapt getBottom to
use the precision values as well.

Author: akoch-yatta

org.eclipse.draw2d.tests/src/org/eclipse/draw2d/test/PrecisionRectangleTest.java

@@ -174,4 +174,19 @@ public void testGetBottomRight() {
 		assertEquals(rect.getBottomRight().y - rect.getTopLeft().y, rect.height);
 	}
 
+	@SuppressWarnings("static-method")
+	@Test
+	public void testConsistencyGetBottom() {
+		Rectangle rect = new PrecisionRectangle(100.5, 100.5, 250.5, 250.5);
+		assertEquals(rect.getBottomRight().y, rect.getBottom().y);
+		assertEquals(rect.getBottomLeft().y, rect.getBottom().y);
+	}
+
+	@SuppressWarnings("static-method")
+	@Test
+	public void testConsistencyGetTop() {
+		Rectangle rect = new PrecisionRectangle(100.5, 100.5, 250.5, 250.5);
+		assertEquals(rect.getTopRight().y, rect.getTop().y);
+		assertEquals(rect.getTopLeft().y, rect.getTop().y);
+	}
 }

org.eclipse.draw2d/src/org/eclipse/draw2d/geometry/PrecisionRectangle.java

@@ -225,7 +225,7 @@ private PrecisionRectangle expandPrecise(double h, double v) {
 	 */
 	@Override
 	public Point getBottom() {
-		return new PrecisionPoint(preciseX() + preciseWidth() / 2, bottom());
+		return new PrecisionPoint(preciseX() + preciseWidth() / 2, preciseY() + preciseHeight());
 	}
 
 	/**