You are viewing a plain text version of this content. The canonical link for it is here.
Posted to fop-commits@xmlgraphics.apache.org by vh...@apache.org on 2014/04/30 20:20:40 UTC
svn commit: r1591437 - in
/xmlgraphics/fop/branches/Temp_WhitespaceManagement: ./
src/java/org/apache/fop/pdf/ src/java/org/apache/fop/render/bitmap/
src/java/org/apache/fop/render/pdf/ src/java/org/apache/fop/render/ps/
src/java/org/apache/fop/svg/ te...
Author: vhennebert
Date: Wed Apr 30 18:20:40 2014
New Revision: 1591437
URL: http://svn.apache.org/r1591437
Log:
Brought the branch in sync with rev. 1590916 of trunk
Added:
xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/pdf/TransparencyDisallowedException.java
- copied unchanged from r1591423, xmlgraphics/fop/trunk/src/java/org/apache/fop/pdf/TransparencyDisallowedException.java
xmlgraphics/fop/branches/Temp_WhitespaceManagement/test/xml/pdf-a/svg-with-transparent-image.fo
- copied unchanged from r1591423, xmlgraphics/fop/trunk/test/xml/pdf-a/svg-with-transparent-image.fo
Modified:
xmlgraphics/fop/branches/Temp_WhitespaceManagement/ (props changed)
xmlgraphics/fop/branches/Temp_WhitespaceManagement/findbugs-exclude.xml
xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/pdf/PDFProfile.java
xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/pdf/PDFResources.java
xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/bitmap/AbstractBitmapDocumentHandler.java
xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/pdf/PDFContentGenerator.java
xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/pdf/PDFImageHandlerSVG.java
xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/ps/PSImageHandlerRenderedImage.java
xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/svg/AbstractFOPTextPainter.java
xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/svg/SVGEventProducer.java
xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/svg/SVGEventProducer.xml
xmlgraphics/fop/branches/Temp_WhitespaceManagement/test/java/org/apache/fop/events/EventChecker.java
xmlgraphics/fop/branches/Temp_WhitespaceManagement/test/java/org/apache/fop/render/pdf/PDFAConformanceTestCase.java
Propchange: xmlgraphics/fop/branches/Temp_WhitespaceManagement/
------------------------------------------------------------------------------
Merged /xmlgraphics/fop/trunk:r1578271-1591423
Modified: xmlgraphics/fop/branches/Temp_WhitespaceManagement/findbugs-exclude.xml
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_WhitespaceManagement/findbugs-exclude.xml?rev=1591437&r1=1591436&r2=1591437&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_WhitespaceManagement/findbugs-exclude.xml (original)
+++ xmlgraphics/fop/branches/Temp_WhitespaceManagement/findbugs-exclude.xml Wed Apr 30 18:20:40 2014
@@ -444,6 +444,21 @@
<Method name="renderSVGDocument"/>
</Or>
</And>
+ <And>
+ <Class name="org.apache.fop.render.ps.PSTextPainter"/>
+ <Or>
+ <Field name="gen"/>
+ <Field name="ps"/>
+ <Field name="psRun"/>
+ <Field name="textUtil"/>
+ </Or>
+ </And>
+ <And>
+ <Class name="org.apache.fop.svg.PDFTextPainter"/>
+ <Or>
+ <Field name="pdf"/>
+ </Or>
+ </And>
</Or>
</Match>
<Match>
Modified: xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/pdf/PDFProfile.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/pdf/PDFProfile.java?rev=1591437&r1=1591436&r2=1591437&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/pdf/PDFProfile.java (original)
+++ xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/pdf/PDFProfile.java Wed Apr 30 18:20:40 2014
@@ -171,8 +171,7 @@ public class PDFProfile {
public void verifyTransparencyAllowed(String context) {
Object profile = isTransparencyAllowed();
if (profile != null) {
- throw new PDFConformanceException(profile + " does not allow the use of transparency. ("
- + profile + ")");
+ throw new TransparencyDisallowedException(profile, context);
}
}
Modified: xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/pdf/PDFResources.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/pdf/PDFResources.java?rev=1591437&r1=1591436&r2=1591437&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/pdf/PDFResources.java (original)
+++ xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/pdf/PDFResources.java Wed Apr 30 18:20:40 2014
@@ -45,7 +45,7 @@ public class PDFResources extends PDFDic
/**
* /Font objects keyed by their internal name
*/
- protected Map<String, PDFFont> fonts = new LinkedHashMap<String, PDFFont>();
+ protected Map<String, PDFDictionary> fonts = new LinkedHashMap<String, PDFDictionary>();
/**
* Set of XObjects
@@ -76,6 +76,8 @@ public class PDFResources extends PDFDic
/** Named properties */
protected Map<String, PDFReference> properties = new LinkedHashMap<String, PDFReference>();
+ private PDFResources parent;
+
/**
* create a /Resources object.
*
@@ -87,13 +89,29 @@ public class PDFResources extends PDFDic
setObjectNumber(objnum);
}
+ public void setParentResources(PDFResources p) {
+ parent = p;
+ }
+
+ public PDFResources getParentResources() {
+ return parent;
+ }
+
/**
* add font object to resources list.
*
* @param font the PDFFont to add
*/
public void addFont(PDFFont font) {
- this.fonts.put(font.getName(), font);
+ fonts.put(font.getName(), font);
+ }
+
+ public void addFont(String name, PDFDictionary font) {
+ fonts.put(name, font);
+ }
+
+ public Map<String, PDFDictionary> getFonts() {
+ return fonts;
}
/**
@@ -220,12 +238,17 @@ public class PDFResources extends PDFDic
}
private void populateDictionary() {
- if (!this.fonts.isEmpty()) {
+ if (!this.fonts.isEmpty() || (parent != null && !parent.getFonts().isEmpty())) {
PDFDictionary dict = new PDFDictionary(this);
/* construct PDF dictionary of font object references */
- for (Map.Entry<String, PDFFont> entry : fonts.entrySet()) {
+ for (Map.Entry<String, PDFDictionary> entry : fonts.entrySet()) {
dict.put(entry.getKey(), entry.getValue());
}
+ if (parent != null) {
+ for (Map.Entry<String, PDFDictionary> entry : parent.getFonts().entrySet()) {
+ dict.put(entry.getKey(), entry.getValue());
+ }
+ }
put("Font", dict);
}
Modified: xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/bitmap/AbstractBitmapDocumentHandler.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/bitmap/AbstractBitmapDocumentHandler.java?rev=1591437&r1=1591436&r2=1591437&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/bitmap/AbstractBitmapDocumentHandler.java (original)
+++ xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/bitmap/AbstractBitmapDocumentHandler.java Wed Apr 30 18:20:40 2014
@@ -229,7 +229,10 @@ public abstract class AbstractBitmapDocu
}
//Set up bitmap to paint on
- this.currentImage = createBufferedImage(bitmapWidth, bitmapHeight);
+ if (currentImage == null || currentImage.getWidth() != bitmapWidth
+ || currentImage.getHeight() != bitmapHeight) {
+ currentImage = createBufferedImage(bitmapWidth, bitmapHeight);
+ }
Graphics2D graphics2D = this.currentImage.createGraphics();
// draw page background
@@ -316,7 +319,6 @@ public abstract class AbstractBitmapDocu
this.multiImageWriter.writeImage(this.currentImage,
getSettings().getWriterParams());
}
- this.currentImage = null;
} catch (IOException ioe) {
throw new IFException("I/O error while encoding BufferedImage", ioe);
}
Modified: xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/pdf/PDFContentGenerator.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/pdf/PDFContentGenerator.java?rev=1591437&r1=1591436&r2=1591437&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/pdf/PDFContentGenerator.java (original)
+++ xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/pdf/PDFContentGenerator.java Wed Apr 30 18:20:40 2014
@@ -486,6 +486,13 @@ public class PDFContentGenerator {
restoreGraphicsState();
}
+ public void placeImage(AffineTransform at, String stream) {
+ saveGraphicsState();
+ concatenate(at);
+ add(stream);
+ restoreGraphicsState();
+ }
+
/**
* Places a previously registered image at a certain place on the page,
* bracketing it as a marked-content sequence.
Modified: xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/pdf/PDFImageHandlerSVG.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/pdf/PDFImageHandlerSVG.java?rev=1591437&r1=1591436&r2=1591437&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/pdf/PDFImageHandlerSVG.java (original)
+++ xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/pdf/PDFImageHandlerSVG.java Wed Apr 30 18:20:40 2014
@@ -44,6 +44,7 @@ import org.apache.fop.apps.FOUserAgent;
import org.apache.fop.events.EventBroadcaster;
import org.apache.fop.image.loader.batik.BatikImageFlavors;
import org.apache.fop.image.loader.batik.BatikUtil;
+import org.apache.fop.pdf.TransparencyDisallowedException;
import org.apache.fop.render.ImageHandler;
import org.apache.fop.render.ImageHandlerUtil;
import org.apache.fop.render.RenderingContext;
@@ -207,6 +208,10 @@ public class PDFImageHandlerSVG implemen
root.paint(graphics);
ctx.dispose();
generator.add(graphics.getString());
+ } catch (TransparencyDisallowedException e) {
+ SVGEventProducer eventProducer = SVGEventProducer.Provider.get(
+ context.getUserAgent().getEventBroadcaster());
+ eventProducer.bitmapWithTransparency(this, e.getProfile(), image.getInfo().getOriginalURI());
} catch (Exception e) {
SVGEventProducer eventProducer = SVGEventProducer.Provider.get(
context.getUserAgent().getEventBroadcaster());
Modified: xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/ps/PSImageHandlerRenderedImage.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/ps/PSImageHandlerRenderedImage.java?rev=1591437&r1=1591436&r2=1591437&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/ps/PSImageHandlerRenderedImage.java (original)
+++ xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/render/ps/PSImageHandlerRenderedImage.java Wed Apr 30 18:20:40 2014
@@ -19,9 +19,12 @@
package org.apache.fop.render.ps;
+import java.awt.Color;
import java.awt.Dimension;
+import java.awt.Graphics2D;
import java.awt.Rectangle;
import java.awt.geom.Rectangle2D;
+import java.awt.image.BufferedImage;
import java.awt.image.ColorModel;
import java.awt.image.RenderedImage;
import java.io.IOException;
@@ -63,6 +66,17 @@ public class PSImageHandlerRenderedImage
Rectangle2D targetRect = new Rectangle2D.Double(x, y, w, h);
RenderedImage ri = imageRend.getRenderedImage();
+
+ if (ri instanceof BufferedImage && ri.getColorModel().hasAlpha()) {
+ BufferedImage convertedImg = new BufferedImage(ri.getWidth(), ri.getHeight(), BufferedImage.TYPE_INT_RGB);
+ Graphics2D g = (Graphics2D) convertedImg.getGraphics();
+ g.setBackground(Color.WHITE);
+ g.clearRect(0, 0, ri.getWidth(), ri.getHeight());
+ g.drawImage((BufferedImage)ri, 0, 0, null);
+ g.dispose();
+ ri = convertedImg;
+ }
+
ImageEncoder encoder = ImageEncodingHelper.createRenderedImageEncoder(ri);
Dimension imgDim = new Dimension(ri.getWidth(), ri.getHeight());
String imgDescription = ri.getClass().getName();
Modified: xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/svg/AbstractFOPTextPainter.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/svg/AbstractFOPTextPainter.java?rev=1591437&r1=1591436&r2=1591437&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/svg/AbstractFOPTextPainter.java (original)
+++ xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/svg/AbstractFOPTextPainter.java Wed Apr 30 18:20:40 2014
@@ -152,8 +152,21 @@ public abstract class AbstractFOPTextPai
double y = outputLocation.getY();
try {
try {
+ AFPGraphics2D afpg2d = (AFPGraphics2D)g2d;
+ int fontSize = 0;
+ if (font != null) {
+ fontSize = (int) Math.round(afpg2d.convertToAbsoluteLength(font.getFontSize()));
+ }
+ if (fontSize < 6000) {
+ nativeTextHandler.drawString(g2d, txt, (float) (x + tx), (float) y);
+ } else {
+ double scaleX = g2d.getTransform().getScaleX();
+ for (int i = 0; i < txt.length(); i++) {
+ double ad = run.getLayout().getGlyphAdvances()[i] * scaleX;
+ nativeTextHandler.drawString(g2d, txt.charAt(i) + "", (float) (x + tx + ad), (float) y);
+ }
+ }
//TODO draw underline and overline if set
- nativeTextHandler.drawString(g2d, txt, (float) (x + tx), (float) y);
//TODO draw strikethrough if set
} catch (IOException ioe) {
if (g2d instanceof AFPGraphics2D) {
Modified: xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/svg/SVGEventProducer.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/svg/SVGEventProducer.java?rev=1591437&r1=1591436&r2=1591437&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/svg/SVGEventProducer.java (original)
+++ xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/svg/SVGEventProducer.java Wed Apr 30 18:20:40 2014
@@ -98,4 +98,13 @@ public interface SVGEventProducer extend
*/
void transparencyIgnored(Object source, Object pdfProfile, String uri);
+
+ /**
+ * SVG references a bitmap image that contains transparency while it is not allowed.
+ * @param source the event source
+ * @param pdfProfile the PDF profile that disallows transparency
+ * @param uri the image URI, if available
+ * @event.severity ERROR
+ */
+ void bitmapWithTransparency(Object source, Object pdfProfile, String uri);
}
Modified: xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/svg/SVGEventProducer.xml
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/svg/SVGEventProducer.xml?rev=1591437&r1=1591436&r2=1591437&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/svg/SVGEventProducer.xml (original)
+++ xmlgraphics/fop/branches/Temp_WhitespaceManagement/src/java/org/apache/fop/svg/SVGEventProducer.xml Wed Apr 30 18:20:40 2014
@@ -23,4 +23,5 @@
<message key="svgNotBuilt">SVG graphic could not be built. Reason: {e}</message>
<message key="svgRenderingError">SVG graphic could not be rendered. Reason: {e}</message>
<message key="transparencyIgnored">Transparency in an SVG image will be ignored because {pdfProfile} does not allow it[ (see {uri})].</message>
+ <message key="bitmapWithTransparency">An SVG image is referencing a bitmap with transparency while {pdfProfile} does not allow it[ (see {uri})].</message>
</catalogue>
Modified: xmlgraphics/fop/branches/Temp_WhitespaceManagement/test/java/org/apache/fop/events/EventChecker.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_WhitespaceManagement/test/java/org/apache/fop/events/EventChecker.java?rev=1591437&r1=1591436&r2=1591437&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_WhitespaceManagement/test/java/org/apache/fop/events/EventChecker.java (original)
+++ xmlgraphics/fop/branches/Temp_WhitespaceManagement/test/java/org/apache/fop/events/EventChecker.java Wed Apr 30 18:20:40 2014
@@ -46,7 +46,7 @@ public class EventChecker implements Eve
if (id.equals(expectedEventID)) {
eventReceived = true;
for (Map.Entry<String, Object> param : expectedParams.entrySet()) {
- assertEquals(event.getParam(param.getKey()), param.getValue());
+ assertEquals(param.getValue(), event.getParam(param.getKey()));
}
}
}
Modified: xmlgraphics/fop/branches/Temp_WhitespaceManagement/test/java/org/apache/fop/render/pdf/PDFAConformanceTestCase.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_WhitespaceManagement/test/java/org/apache/fop/render/pdf/PDFAConformanceTestCase.java?rev=1591437&r1=1591436&r2=1591437&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_WhitespaceManagement/test/java/org/apache/fop/render/pdf/PDFAConformanceTestCase.java (original)
+++ xmlgraphics/fop/branches/Temp_WhitespaceManagement/test/java/org/apache/fop/render/pdf/PDFAConformanceTestCase.java Wed Apr 30 18:20:40 2014
@@ -115,15 +115,29 @@ public class PDFAConformanceTestCase ext
@Test
public void svgTransparency() throws Exception {
- Map<String, Object> params = new HashMap<String, Object>();
- params.put("pdfProfile", PDFAMode.PDFA_1B);
- EventChecker eventChecker = new EventChecker(SVGEventProducer.class.getName()
- + ".transparencyIgnored", params);
FOUserAgent ua = getUserAgent();
- ua.getEventBroadcaster().addEventListener(eventChecker);
+ EventChecker eventChecker = setupEventChecker(ua, "transparencyIgnored");
File foFile = new File(foBaseDir, "svg-transparency.fo");
convertFO(foFile, ua, dumpPDF);
eventChecker.end();
}
+ @Test
+ public void svgContainingBitmapWithTransparency() throws Exception {
+ FOUserAgent ua = getUserAgent();
+ EventChecker eventChecker = setupEventChecker(ua, "bitmapWithTransparency");
+ File foFile = new File(foBaseDir, "svg-with-transparent-image.fo");
+ convertFO(foFile, ua, dumpPDF);
+ eventChecker.end();
+ }
+
+ private EventChecker setupEventChecker(FOUserAgent ua, String expectedEvent) {
+ Map<String, Object> params = new HashMap<String, Object>();
+ params.put("pdfProfile", PDFAMode.PDFA_1B);
+ EventChecker eventChecker = new EventChecker(SVGEventProducer.class.getName()
+ + "." + expectedEvent, params);
+ ua.getEventBroadcaster().addEventListener(eventChecker);
+ return eventChecker;
+ }
+
}
---------------------------------------------------------------------
To unsubscribe, e-mail: fop-commits-unsubscribe@xmlgraphics.apache.org
For additional commands, e-mail: fop-commits-help@xmlgraphics.apache.org