From 809b38a9c2e5537b4a532a4680f7fc06d8c7dbb2 Mon Sep 17 00:00:00 2001 From: Sravan Kumar Lakkimsetti Date: Mon, 21 May 2018 13:54:07 +0530 Subject: Bug 534817 - screenshots do not change Change-Id: I8cf7d72ab79e6dc04a456da999c8901cf2f4d2a6 Signed-off-by: Sravan Kumar Lakkimsetti --- .../Eclipse SWT/gtk/org/eclipse/swt/graphics/Image.java | 11 ++++------- .../Eclipse SWT/gtk/org/eclipse/swt/internal/ImageList.java | 10 ++++++++-- 2 files changed, 12 insertions(+), 9 deletions(-) diff --git a/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/graphics/Image.java b/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/graphics/Image.java index 8301156deb..1c31c6dc61 100644 --- a/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/graphics/Image.java +++ b/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/graphics/Image.java @@ -1390,18 +1390,15 @@ void init(int width, int height) { this.type = SWT.BITMAP; /* Create the pixmap */ - if (GTK.GTK3) { - surface = Cairo.cairo_image_surface_create(Cairo.CAIRO_FORMAT_ARGB32, width, height); - } else { - surface = GDK.gdk_window_create_similar_surface(GDK.gdk_get_default_root_window(), Cairo.CAIRO_CONTENT_COLOR, width, height); - } + surface = GDK.gdk_window_create_similar_surface(GDK.gdk_get_default_root_window(), Cairo.CAIRO_CONTENT_COLOR, width, height); if (surface == 0) SWT.error(SWT.ERROR_NO_HANDLES); - // When we create a blank image we need to set it to 100 in GTK3 as we don't scale here. - // Cairo will scale for us + // When we create a blank image we need to set it to 100 in GTK3 as we draw using 100% scale. + // Cairo will take care of scaling for us when image needs to be scaled. if (!GTK.GTK3) { currentDeviceZoom = DPIUtil.getDeviceZoom(); } else { currentDeviceZoom = 100; + Cairo.cairo_surface_set_device_scale(surface, 1f, 1f); } long /*int*/ cairo = Cairo.cairo_create(surface); if (cairo == 0) SWT.error(SWT.ERROR_NO_HANDLES); diff --git a/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/internal/ImageList.java b/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/internal/ImageList.java index a611073ea1..0f38b3fec0 100644 --- a/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/internal/ImageList.java +++ b/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/internal/ImageList.java @@ -34,9 +34,15 @@ public static long /*int*/ convertSurface(Image image) { int format = Cairo.cairo_surface_get_content(newSurface) == Cairo.CAIRO_CONTENT_COLOR ? Cairo.CAIRO_FORMAT_RGB24 : Cairo.CAIRO_FORMAT_ARGB32; newSurface = Cairo.cairo_image_surface_create(format, bounds.width, bounds.height); if (newSurface == 0) SWT.error(SWT.ERROR_NO_HANDLES); + //retain device scale set in the original surface if (GTK.GTK3) { - double scaleFactor = DPIUtil.getDeviceZoom() / 100f; - Cairo.cairo_surface_set_device_scale(newSurface, scaleFactor, scaleFactor); + double sx[] = new double[1]; + double sy[] = new double[1]; + Cairo.cairo_surface_get_device_scale(image.surface, sx, sy); + if (sx[0] == 0 || sy[0] == 0){ + sx[0] = sy[0] = DPIUtil.getDeviceZoom() / 100f; + } + Cairo.cairo_surface_set_device_scale(newSurface, sx[0], sy[0]); } long /*int*/ cairo = Cairo.cairo_create(newSurface); if (cairo == 0) SWT.error(SWT.ERROR_NO_HANDLES); -- cgit v1.2.3