Skip to content

Commit cddfab9

Browse files
committed
[eRCP] Remove obsolete ImageSupport class
Primarily used in the ImageEvaluator to create an Image from e.g. the file system. Contributes to #942
1 parent c8f13a8 commit cddfab9

File tree

4 files changed

+6
-116
lines changed

4 files changed

+6
-116
lines changed

org.eclipse.wb.swt/src/org/eclipse/wb/internal/swt/model/property/editor/image/ImageEvaluator.java

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -14,19 +14,20 @@
1414
import org.eclipse.wb.core.eval.EvaluationContext;
1515
import org.eclipse.wb.core.eval.IExpressionEvaluator;
1616
import org.eclipse.wb.core.model.IGenericProperty;
17+
import org.eclipse.wb.internal.core.DesignerPlugin;
1718
import org.eclipse.wb.internal.core.model.JavaInfoEvaluationHelper;
1819
import org.eclipse.wb.internal.core.utils.ast.AstNodeUtils;
1920
import org.eclipse.wb.internal.core.utils.ast.DomGenerics;
20-
import org.eclipse.wb.internal.core.utils.reflect.ReflectionUtils;
2121
import org.eclipse.wb.internal.swt.model.property.editor.image.plugin.WorkspacePluginInfo;
22-
import org.eclipse.wb.internal.swt.support.ImageSupport;
2322

2423
import org.eclipse.core.resources.IFile;
2524
import org.eclipse.core.resources.IProject;
2625
import org.eclipse.core.resources.ResourcesPlugin;
2726
import org.eclipse.jdt.core.dom.Expression;
2827
import org.eclipse.jdt.core.dom.ITypeBinding;
2928
import org.eclipse.jdt.core.dom.MethodInvocation;
29+
import org.eclipse.jface.resource.ImageDescriptor;
30+
import org.eclipse.swt.graphics.Image;
3031
import org.eclipse.ui.plugin.AbstractUIPlugin;
3132

3233
import java.io.InputStream;
@@ -98,14 +99,14 @@ private static Object getPluginImage(MethodInvocation invocation,
9899
// prepare image path
99100
String imagePath = (String) AstEvaluationEngine.evaluate(context, arguments.get(1));
100101
// load image
101-
Object image;
102+
Image image;
102103
try {
103104
InputStream stream = provider.getInputStream(imagePath);
104105
if (stream == null) {
105106
return null;
106107
}
107108
try {
108-
image = ImageSupport.createImage(stream);
109+
image = new Image(DesignerPlugin.getStandardDisplay(), stream);
109110
} finally {
110111
stream.close();
111112
}
@@ -117,12 +118,7 @@ private static Object getPluginImage(MethodInvocation invocation,
117118
return image;
118119
}
119120
// handle for ResourceManager.getPluginImageDescriptor()
120-
Class<?> imageDescriptorClass =
121-
context.getClassLoader().loadClass("org.eclipse.jface.resource.ImageDescriptor");
122-
return ReflectionUtils.invokeMethod(
123-
imageDescriptorClass,
124-
"createFromImage(org.eclipse.swt.graphics.Image)",
125-
image);
121+
return ImageDescriptor.createFromImage(image);
126122
}
127123

128124
private static interface InputStreamProvider {

org.eclipse.wb.swt/src/org/eclipse/wb/internal/swt/support/ImageSupport.java

Lines changed: 0 additions & 52 deletions
This file was deleted.

org.eclipse.wb.tests/src/org/eclipse/wb/tests/designer/swt/support/ImageSupportTest.java

Lines changed: 0 additions & 53 deletions
This file was deleted.

org.eclipse.wb.tests/src/org/eclipse/wb/tests/designer/swt/support/SupportTests.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424
CoordinateUtilsTest.class,
2525
ColorSupportTest.class,
2626
FontSupportTest.class,
27-
ImageSupportTest.class,
2827
RowLayoutSupportTest.class,
2928
ControlSupportTest.class,
3029
ContainerSupportTest.class

0 commit comments

Comments
 (0)