From e9f7fa3fa3467032f9741937c40d34f25556399d Mon Sep 17 00:00:00 2001 From: Shahzaib Ibrahim Date: Wed, 10 Dec 2025 14:21:48 +0100 Subject: [PATCH] Removing usages of Display#getDPI from test Having the scale factor being based on the screen DPI leads to unexpected result e.g. Image too big/small. Having a screen dpi independent factor leads to consistent results --- .../tests/gtk/snippets/Bug507020_WaylandIconsDoubleScaled.java | 2 +- .../swt/tests/junit/Test_org_eclipse_swt_widgets_Display.java | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/gtk/snippets/Bug507020_WaylandIconsDoubleScaled.java b/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/gtk/snippets/Bug507020_WaylandIconsDoubleScaled.java index 49bb534ffa5..fce8c053571 100644 --- a/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/gtk/snippets/Bug507020_WaylandIconsDoubleScaled.java +++ b/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/gtk/snippets/Bug507020_WaylandIconsDoubleScaled.java @@ -42,7 +42,7 @@ public static void main (String [] args) { gridData.horizontalSpan = 2; canvas.setLayoutData (gridData); - new Label (shell, SWT.NONE).setText ("5. 50x50 box\n(Display#getDPI(): " + display.getDPI().x + ")"); + new Label (shell, SWT.NONE).setText ("5. 50x50 box"); Label box= new Label (shell, SWT.NONE); box.setBackground(display.getSystemColor(SWT.COLOR_WIDGET_DARK_SHADOW)); box.setLayoutData (new GridData (50, 50)); diff --git a/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_widgets_Display.java b/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_widgets_Display.java index 11a67f30d08..91610a25577 100644 --- a/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_widgets_Display.java +++ b/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_widgets_Display.java @@ -1553,6 +1553,7 @@ public void test_wake() { /* custom */ boolean disposeExecRan; +@SuppressWarnings("deprecation") @Test public void test_getDPI() { Display display = new Display();