[335892] enable to highlight target element in LV mode
diff --git a/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/LowVisionProblemConverter.java b/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/LowVisionProblemConverter.java index fd8cf16..a270658 100644 --- a/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/LowVisionProblemConverter.java +++ b/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/LowVisionProblemConverter.java
@@ -35,7 +35,7 @@ if (type == LowVisionProblem.LOWVISION_COLOR_PROBLEM) { cp = (ColorProblem) target[i].getRepresentative(); tmp = new ProblemItemLV( - "L_" + target[i].getLowVisionProblemType()+"."+cp.getLevel()); //$NON-NLS-1$ + "L_" + target[i].getLowVisionProblemType() + "." + cp.getLevel()); //$NON-NLS-1$ tmp.setTargetNode(cp.getElement()); tmp.setTargetString(cp.getAdditionalDescription()); } else { @@ -45,15 +45,18 @@ tmp.setSubType(type); try { switch (type) { - case ILowvisionProblemSubtype.LOWVISION_BACKGROUND_IMAGE_WARNING: - cp = (ColorProblem)target[i].getRepresentative(); + case ILowvisionProblemSubtype.LOWVISION_BACKGROUND_IMAGE_WARNING: + cp = (ColorProblem) target[i].getRepresentative(); tmp.setTargetNode(cp.getElement()); - tmp.setDescription(tmp.getDescription()+cp.getAdditionalDescription()); + tmp.setTargetString(cp.getAdditionalDescription()); break; case ILowvisionProblemSubtype.LOWVISION_COLOR_PROBLEM: break; default: tmp.setDescription(target[i].getDescription()); + if (target[i].getRepresentative() != null) + tmp.setTargetNode(target[i].getRepresentative() + .getElement()); } } catch (Exception e) { tmp.setDescription("unknown"); //$NON-NLS-1$
diff --git a/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/PageElement.java b/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/PageElement.java index 9d2f9b8..f7e2c4d 100644 --- a/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/PageElement.java +++ b/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/PageElement.java
@@ -220,6 +220,7 @@ FixedSmallFontProblem newProblem = null; try { newProblem = new FixedSmallFontProblem(this, _lvType, proba); + newProblem.setElement(fsfp.getElement()); problemVec.addElement(newProblem); } catch (LowVisionProblemException e) { e.printStackTrace(); @@ -380,8 +381,10 @@ short type = fontSizeType(fontStr); if (type == FONT_SIZE_FIXED) { // not include "pt" try { - return (new FixedSizeFontProblem(this, _lvType, - PageElement.SEVERITY_FIXED_SIZE_FONT)); + FixedSizeFontProblem problem = new FixedSizeFontProblem(this, _lvType, + PageElement.SEVERITY_FIXED_SIZE_FONT); + problem.setElement(style.getElement()); + return (problem); } catch (LowVisionProblemException e) { e.printStackTrace(); return (null); @@ -483,8 +486,10 @@ if (fixedFlag) { try { - return (new FixedSizeFontProblem(this, _lvType, - PageElement.SEVERITY_FIXED_SIZE_FONT)); + FixedSizeFontProblem problem = new FixedSizeFontProblem(this, _lvType, + PageElement.SEVERITY_FIXED_SIZE_FONT); + problem.setElement(style.getElement()); + return (problem); } catch (LowVisionProblemException e) { e.printStackTrace(); return (null); @@ -768,8 +773,10 @@ if (severity > 0.0) { try { // fixed severity - return (new SmallFontProblem(this, _lvType, - PageElement.SEVERITY_SMALL_FONT)); + SmallFontProblem problem = new SmallFontProblem(this, _lvType, + PageElement.SEVERITY_SMALL_FONT); + problem.setElement(style.getElement()); + return (problem); } catch (LowVisionProblemException e) { e.printStackTrace(); return (null);
diff --git a/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/image/InteriorImage.java b/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/image/InteriorImage.java index c3c3a7a..f972211 100644 --- a/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/image/InteriorImage.java +++ b/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/image/InteriorImage.java
@@ -27,6 +27,7 @@ import org.eclipse.actf.visualization.internal.engines.lowvision.problem.LowVisionProblem; import org.eclipse.actf.visualization.internal.engines.lowvision.problem.LowVisionProblemException; import org.eclipse.actf.visualization.internal.engines.lowvision.problem.LowVisionProblemGroup; +import org.w3c.dom.Element; public class InteriorImage extends PageComponent { @@ -53,6 +54,16 @@ InteriorImageComponent[] largeComponents = null; String url = null; + + Element image; + + public Element getImage() { + return image; + } + + public void setImageElement(Element image) { + this.image = image; + } // a part of PageImage public InteriorImage(PageImage _pi, int _x, int _y, int _width, int _height) { @@ -244,6 +255,7 @@ ImageColorProblem probl = new ImageColorProblem(this, _lvType, probability, iic1, iic2); + probl.setElement(image); problemVector.addElement(probl); } }
diff --git a/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/image/PageImage.java b/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/image/PageImage.java index 1bc926e..6d7aaa6 100644 --- a/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/image/PageImage.java +++ b/plugins/org.eclipse.actf.visualization.engines.lowvision/src/org/eclipse/actf/visualization/internal/engines/lowvision/image/PageImage.java
@@ -276,6 +276,7 @@ } InteriorImage curIm = new InteriorImage(this, curPos); + curIm.setImageElement(curPos.getElement()); imageVector.addElement(curIm); } int size = imageVector.size(); @@ -854,6 +855,7 @@ return (null); } + @SuppressWarnings("unused") public List<IProblemItem> checkCharacters(LowVisionType _lvType, String urlS, int frameId) throws ImageException, LowVisionProblemException {