Fix replaceTextRange logic for unfocused StyledText with variable line

height

- Updated productive code to prevent IllegalArgumentException when
replacing text in a StyledText control with variable line height and no
focus, by adding boundary checks.
- Added a regression test to validate the fix: ensures no exception is
thrown when replacing the full text range under these conditions.

Fixes: #2302

Author: tobias-melcher

bundles/org.eclipse.swt/Eclipse SWT Custom Widgets/common/org/eclipse/swt/custom/StyledText.java

@@ -1077,31 +1077,50 @@ void calculateTopIndex(int delta) {
 			}
 		}
 	} else {
+		int lineCount = content.getLineCount();
 		if (delta >= 0) {
 			delta -= topIndexY;
 			int lineIndex = topIndex;
-			int lineCount = content.getLineCount();
 			while (lineIndex < lineCount) {
 				if (delta <= 0) break;
-				delta -= renderer.getCachedLineHeight(lineIndex++);
+				if (lineIndex >= 0 && lineIndex < lineCount) {
+					delta -= renderer.getCachedLineHeight(lineIndex);
+				}
+				lineIndex++;
+			}
+			int lineHeight = 0;
+			if (lineIndex >= 0 && lineIndex < lineCount) {
+				lineHeight = renderer.getCachedLineHeight(lineIndex);
 			}
-			if (lineIndex < lineCount && -delta + renderer.getCachedLineHeight(lineIndex) <= clientAreaHeight - topMargin - bottomMargin) {
+			if (lineIndex < lineCount && -delta + lineHeight <= clientAreaHeight - topMargin - bottomMargin) {
 				topIndex = lineIndex;
 				topIndexY = -delta;
 			} else {
 				topIndex = lineIndex - 1;
-				topIndexY = -renderer.getCachedLineHeight(topIndex) - delta;
+				int topIndexHeight = 0;
+				if (topIndex >= 0 && topIndex < lineCount) {
+					topIndexHeight = renderer.getCachedLineHeight(topIndex);
+				}
+				topIndexY = -topIndexHeight - delta;
 			}
 		} else {
 			delta -= topIndexY;
 			int lineIndex = topIndex;
 			while (lineIndex > 0) {
-				int lineHeight = renderer.getCachedLineHeight(lineIndex - 1);
+				int previousLineIndex = lineIndex - 1;
+				int lineHeight = 0;
+				if (previousLineIndex >= 0 && previousLineIndex < lineCount) {
+					lineHeight = renderer.getCachedLineHeight(previousLineIndex);
+				}
 				if (delta + lineHeight > 0) break;
 				delta += lineHeight;
 				lineIndex--;
 			}
-			if (lineIndex == 0 || -delta + renderer.getCachedLineHeight(lineIndex) <= clientAreaHeight - topMargin - bottomMargin) {
+			int lineHeight = 0;
+			if (lineIndex >= 0 && lineIndex < lineCount) {
+				lineHeight = renderer.getCachedLineHeight(lineIndex);
+			}
+			if (lineIndex == 0 || -delta + lineHeight <= clientAreaHeight - topMargin - bottomMargin) {
 				topIndex = lineIndex;
 				topIndexY = - delta;
 			} else {
@@ -1381,8 +1400,12 @@ int getAvailableHeightAbove(int height) {
 		if (topIndexY > 0) {
 			maxHeight += renderer.getLineHeight(lineIndex--);
 		}
+		int lineCount=content.getLineCount();
 		while (height > maxHeight && lineIndex >= 0) {
-			maxHeight += renderer.getLineHeight(lineIndex--);
+			if (lineIndex>=0 && lineIndex<lineCount) {
+				maxHeight += renderer.getLineHeight(lineIndex);
+			}
+			lineIndex--;
 		}
 	}
 	return Math.min(height, maxHeight);
@@ -3877,10 +3900,16 @@ public int getLinePixel(int lineIndex) {
 	int height = topIndexY;
 	if (lineIndex > topIndex) {
 		for (int i = topIndex; i < lineIndex; i++) {
+			if (i < 0 || i >= lineCount) {
+				continue;
+			}
 			height += renderer.getLineHeight(i);
 		}
 	} else {
 		for (int i = topIndex - 1; i >= lineIndex; i--) {
+			if (i < 0 || i >= lineCount) {
+				continue;
+			}
 			height -= renderer.getLineHeight(i);
 		}
 	}
@@ -3914,10 +3943,18 @@ public int getLineIndex(int y) {
 		}
 	} else {
 		int lineCount = content.getLineCount();
-		int lineHeight = renderer.getLineHeight(line);
+		int lineHeight = 0;
+		if (line>=0 && line<lineCount) {
+			lineHeight = renderer.getLineHeight(line);
+		}
 		while (y - lineHeight >= topIndexY && line < lineCount - 1) {
 			y -= lineHeight;
-			lineHeight = renderer.getLineHeight(++line);
+			++line;
+			if (line >= 0 && line < lineCount) {
+				lineHeight = renderer.getLineHeight(line);
+			}else {
+				lineHeight = 0;
+			}
 		}
 	}
 	return line;

tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_custom_StyledText.java

@@ -70,13 +70,15 @@
 import org.eclipse.swt.graphics.RGB;
 import org.eclipse.swt.graphics.Rectangle;
 import org.eclipse.swt.internal.BidiUtil;
+import org.eclipse.swt.layout.FillLayout;
 import org.eclipse.swt.layout.GridData;
 import org.eclipse.swt.layout.GridLayout;
 import org.eclipse.swt.printing.Printer;
 import org.eclipse.swt.widgets.Caret;
 import org.eclipse.swt.widgets.Display;
 import org.eclipse.swt.widgets.Event;
 import org.eclipse.swt.widgets.ScrollBar;
+import org.eclipse.swt.widgets.Text;
 import org.eclipse.swt.widgets.Widget;
 import org.junit.Assume;
 import org.junit.Before;
@@ -195,6 +197,36 @@ public void test_getTextBounds() {
 	}
 }
 
+@Test
+public void test_replaceTextRangeWithVariableHeight() {
+	shell.setLayout(new FillLayout());
+	StyledText styledText = new StyledText(shell, SWT.BORDER | SWT.V_SCROLL);
+	Text text = new Text(shell, SWT.BORDER);
+	try {
+		String lines = IntStream.range(0, 10)
+				.collect(StringBuilder::new, (s, i) -> s.append("line " + (i + 1) + "\n"), StringBuilder::append).toString();
+		styledText.setText(lines);
+		StyleRange style = new StyleRange();
+		style.start = 0;
+		style.length = lines.length();
+		style.font = styledText.getFont(); // To make the line-height non-fixed
+		styledText.setStyleRange(style);
+
+		shell.setSize(100, 3 * styledText.getLineHeight());
+		shell.open();
+
+		text.setFocus();
+		styledText.setTopIndex(styledText.getLineCount() - 1);
+		assertFalse(styledText.isFocusControl());
+		assertTrue(text.isFocusControl());
+		// ensure no IllegalArgumentException is thrown when styledText control has not the focus and the text is replaced
+		styledText.replaceTextRange(0, styledText.getCharCount(), "");
+	}finally {
+		styledText.dispose();
+		text.dispose();
+	}
+}
+
 @Test
 public void test_addExtendedModifyListenerLorg_eclipse_swt_custom_ExtendedModifyListener() {
 	final String line = "Line1";