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 2009/10/15 16:25:28 UTC

svn commit: r825504 - in /xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop: accessibility/ area/ render/intermediate/ render/pdf/ render/xml/

Author: vhennebert
Date: Thu Oct 15 14:25:28 2009
New Revision: 825504

URL: http://svn.apache.org/viewvc?rev=825504&view=rev
Log:
Merged SimpleStructureTree and ParsedStructureTree classes into a single one, used by both TransformerNodeEndProcessing and intermediate XML parsers

Added:
    xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/StructureTree.java
      - copied, changed from r825461, xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/ParsedStructureTree.java
    xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/StructureTreeBuilder.java   (with props)
Removed:
    xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/ParsedStructureTree.java
    xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/SimpleStructureTree.java
Modified:
    xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/TransformerNodeEndProcessing.java
    xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/area/AreaTreeParser.java
    xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/intermediate/IFParser.java
    xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/intermediate/IFSerializer.java
    xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/pdf/PDFDocumentHandler.java
    xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/pdf/PDFRenderer.java
    xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/xml/XMLRenderer.java

Copied: xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/StructureTree.java (from r825461, xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/ParsedStructureTree.java)
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/StructureTree.java?p2=xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/StructureTree.java&p1=xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/ParsedStructureTree.java&r1=825461&r2=825504&rev=825504&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/ParsedStructureTree.java (original)
+++ xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/StructureTree.java Thu Oct 15 14:25:28 2009
@@ -26,77 +26,53 @@
 import java.util.List;
 
 import javax.xml.transform.Transformer;
-import javax.xml.transform.TransformerConfigurationException;
 import javax.xml.transform.TransformerFactory;
-import javax.xml.transform.dom.DOMResult;
 import javax.xml.transform.dom.DOMSource;
-import javax.xml.transform.sax.SAXTransformerFactory;
-import javax.xml.transform.sax.TransformerHandler;
 import javax.xml.transform.stream.StreamResult;
 
+import org.w3c.dom.Node;
 import org.w3c.dom.NodeList;
-import org.xml.sax.ContentHandler;
-import org.xml.sax.SAXException;
-
-import org.apache.fop.util.DelegatingContentHandler;
 
 /**
- * A StructureTree implementation re-created from the structure stored in an IF
- * XML document.
+ * A reduced version of the document's FO tree, containing only its logical
+ * structure. Used by accessible output formats.
  */
-public class ParsedStructureTree implements StructureTree {
-
-    private SAXTransformerFactory factory;
+public final class StructureTree {
 
-    private List pageSequenceStructures = new ArrayList();
+    private final List pageSequenceStructures = new ArrayList();
 
     /**
-     * Creates a new instance.
-     *
-     * @param factory a factory internally used to build the structures of page
-     * sequences
+     * Package-private default constructor.
      */
-    public ParsedStructureTree(SAXTransformerFactory factory) {
-        this.factory = factory;
-    }
+    StructureTree() { }
 
-    /**
-     * Returns a ContenHandler for parsing the structure of a new page sequence.
-     * It is assumed that page sequences are being parsed in the document order.
-     * This class will automatically number the structure trees.
-     *
-     * @return a handler for parsing the <structure-tree> element and its
-     * descendants
-     * @throws SAXException if there is an error when creating the handler
-     */
-    public ContentHandler getHandlerForNextPageSequence() throws SAXException {
-        TransformerHandler structureTreeBuilder;
-        try {
-            structureTreeBuilder = factory.newTransformerHandler();
-        } catch (TransformerConfigurationException e) {
-            throw new SAXException(e);
-        }
-        final DOMResult domResult = new DOMResult();
-        structureTreeBuilder.setResult(domResult);
-        return new DelegatingContentHandler(structureTreeBuilder) {
-
-            public void characters(char[] ch, int start, int length) throws SAXException {
-                /*
-                 * There's no text node in the structure tree. This is just
-                 * whitespace => ignore
-                 */
+    private static boolean flowOrStaticContentNodes(NodeList nodes) {
+        for (int i = 0; i < nodes.getLength(); i++) {
+            Node node = nodes.item(i);
+            if (node.getNodeType() != Node.ELEMENT_NODE) {
+                return false;
             }
-
-            public void endDocument() throws SAXException {
-                super.endDocument();
-                pageSequenceStructures.add(domResult.getNode().getFirstChild().getChildNodes());
+            String name = node.getLocalName();
+            if (!(name.equals("flow") || name.equals("static-content"))) {
+                return false;
             }
-        };
+        }
+        return true;
+    }
+
+    void addPageSequenceStructure(NodeList structureTree) {
+        assert flowOrStaticContentNodes(structureTree);
+        pageSequenceStructures.add(structureTree);
     }
 
-    /** {@inheritDoc} */
-    public NodeList getPageSequence(int number) {
-        return (NodeList) pageSequenceStructures.get(number - 1);
+    /**
+     * Returns the list of nodes that are the children of the given page sequence.
+     *
+     * @param index index of the page sequence, 0-based
+     * @return its children nodes
+     */
+    public NodeList getPageSequence(int index) {
+        return (NodeList) pageSequenceStructures.get(index);
     }
 
     /**

Added: xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/StructureTreeBuilder.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/StructureTreeBuilder.java?rev=825504&view=auto
==============================================================================
--- xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/StructureTreeBuilder.java (added)
+++ xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/StructureTreeBuilder.java Thu Oct 15 14:25:28 2009
@@ -0,0 +1,95 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+/* $Id$ */
+
+package org.apache.fop.accessibility;
+
+import javax.xml.transform.TransformerConfigurationException;
+import javax.xml.transform.dom.DOMResult;
+import javax.xml.transform.sax.SAXTransformerFactory;
+import javax.xml.transform.sax.TransformerHandler;
+
+import org.xml.sax.ContentHandler;
+import org.xml.sax.SAXException;
+
+import org.apache.fop.util.DelegatingContentHandler;
+
+/**
+ * Helper class that re-builds a structure tree from what is stored in an
+ * intermediate XML file (IF XML or Area Tree XML).
+ */
+public final class StructureTreeBuilder {
+
+    private final SAXTransformerFactory factory;
+
+    private final StructureTree structureTree = new StructureTree();
+
+    /**
+     * Creates a new instance.
+     *
+     * @param factory a factory internally used to build the structures of page
+     * sequences
+     */
+    public StructureTreeBuilder(SAXTransformerFactory factory) {
+        this.factory = factory;
+    }
+
+    /**
+     * Returns the structure tree that will result from the parsing.
+     *
+     * @return the structure tree built by this object
+     */
+    public StructureTree getStructureTree() {
+        return structureTree;
+    }
+
+    /**
+     * Returns a ContenHandler for parsing the structure of a new page sequence.
+     * It is assumed that page sequences are being parsed in the document order.
+     *
+     * @return a handler for parsing the &lt;structure-tree&gt; or
+     * &lt;structureTree&gt; element and its descendants
+     * @throws SAXException if there is an error when creating the handler
+     */
+    public ContentHandler getHandlerForNextPageSequence() throws SAXException {
+        TransformerHandler structureTreeBuilder;
+        try {
+            structureTreeBuilder = factory.newTransformerHandler();
+        } catch (TransformerConfigurationException e) {
+            throw new SAXException(e);
+        }
+        final DOMResult domResult = new DOMResult();
+        structureTreeBuilder.setResult(domResult);
+        return new DelegatingContentHandler(structureTreeBuilder) {
+
+            public void characters(char[] ch, int start, int length) throws SAXException {
+                /*
+                 * There's no text node in the structure tree. This is just
+                 * whitespace => ignore
+                 */
+            }
+
+            public void endDocument() throws SAXException {
+                super.endDocument();
+                structureTree.addPageSequenceStructure(domResult.getNode().getFirstChild()
+                        .getChildNodes());
+            }
+        };
+    }
+
+}

Propchange: xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/StructureTreeBuilder.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/StructureTreeBuilder.java
------------------------------------------------------------------------------
    svn:keywords = Revision Id

Modified: xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/TransformerNodeEndProcessing.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/TransformerNodeEndProcessing.java?rev=825504&r1=825503&r2=825504&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/TransformerNodeEndProcessing.java (original)
+++ xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/accessibility/TransformerNodeEndProcessing.java Thu Oct 15 14:25:28 2009
@@ -32,10 +32,12 @@
 import javax.xml.transform.stream.StreamResult;
 import javax.xml.transform.stream.StreamSource;
 
-import org.apache.commons.io.output.ByteArrayOutputStream;
+import org.w3c.dom.NodeList;
 import org.xml.sax.SAXException;
 import org.xml.sax.helpers.DefaultHandler;
 
+import org.apache.commons.io.output.ByteArrayOutputStream;
+
 import org.apache.fop.apps.FOPException;
 import org.apache.fop.apps.FOUserAgent;
 
@@ -78,7 +80,12 @@
             Source src = new StreamSource(new ByteArrayInputStream(enrichedFO));
             DOMResult res = new DOMResult();
             transformer.transform(src, res);
-            userAgent.setStructureTree(new SimpleStructureTree(res.getNode()));
+            StructureTree structureTree = new StructureTree();
+            NodeList pageSequences = res.getNode().getFirstChild().getChildNodes();
+            for (int i = 0; i < pageSequences.getLength(); i++) {
+                structureTree.addPageSequenceStructure(pageSequences.item(i).getChildNodes());
+            }
+            userAgent.setStructureTree(structureTree);
 
             SAXParserFactory saxParserFactory = SAXParserFactory.newInstance();
             saxParserFactory.setNamespaceAware(true);

Modified: xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/area/AreaTreeParser.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/area/AreaTreeParser.java?rev=825504&r1=825503&r2=825504&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/area/AreaTreeParser.java (original)
+++ xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/area/AreaTreeParser.java Thu Oct 15 14:25:28 2009
@@ -57,7 +57,7 @@
 import org.apache.xmlgraphics.image.loader.ImageSessionContext;
 import org.apache.xmlgraphics.util.QName;
 
-import org.apache.fop.accessibility.ParsedStructureTree;
+import org.apache.fop.accessibility.StructureTreeBuilder;
 import org.apache.fop.apps.FOUserAgent;
 import org.apache.fop.area.Trait.Background;
 import org.apache.fop.area.Trait.InternalLink;
@@ -160,17 +160,17 @@
         private Locator locator;
 
 
-        private ParsedStructureTree structureTree;
+        private StructureTreeBuilder structureTreeBuilder;
 
-        private ContentHandler structureTreeBuilder;
+        private ContentHandler structureTreeBuilderWrapper;
 
-        private final class StructureTreeBuilder extends DelegatingContentHandler {
+        private final class StructureTreeBuilderWrapper extends DelegatingContentHandler {
 
             private Attributes pageSequenceAttributes;
 
-            private StructureTreeBuilder(Attributes pageSequenceAttributes,
-                    ParsedStructureTree structureTree) throws SAXException {
-                super(structureTree.getHandlerForNextPageSequence());
+            private StructureTreeBuilderWrapper(Attributes pageSequenceAttributes)
+                    throws SAXException {
+                super(structureTreeBuilder.getHandlerForNextPageSequence());
                 this.pageSequenceAttributes = new AttributesImpl(pageSequenceAttributes);
             }
 
@@ -216,9 +216,10 @@
             makers.put("bookmarkTree", new BookmarkTreeMaker());
             makers.put("bookmark", new BookmarkMaker());
             makers.put("destination", new DestinationMaker());
+
             if (userAgent.isAccessibilityEnabled()) {
-                structureTree = new ParsedStructureTree(tFactory);
-                userAgent.setStructureTree(structureTree);
+                structureTreeBuilder = new StructureTreeBuilder(tFactory);
+                userAgent.setStructureTree(structureTreeBuilder.getStructureTree());
             }
         }
 
@@ -296,10 +297,10 @@
                 boolean handled = true;
                 if ("".equals(uri)) {
                     if (localName.equals("pageSequence") && userAgent.isAccessibilityEnabled()) {
-                        structureTreeBuilder = new StructureTreeBuilder(attributes, structureTree);
+                        structureTreeBuilderWrapper = new StructureTreeBuilderWrapper(attributes);
                     } else if (localName.equals("structureTree")) {
                         if (userAgent.isAccessibilityEnabled()) {
-                            delegate = structureTreeBuilder;
+                            delegate = structureTreeBuilderWrapper;
                         } else {
                             /* Delegate to a handler that does nothing */
                             delegate = new DefaultHandler();

Modified: xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/intermediate/IFParser.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/intermediate/IFParser.java?rev=825504&r1=825503&r2=825504&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/intermediate/IFParser.java (original)
+++ xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/intermediate/IFParser.java Thu Oct 15 14:25:28 2009
@@ -46,7 +46,7 @@
 
 import org.apache.xmlgraphics.util.QName;
 
-import org.apache.fop.accessibility.ParsedStructureTree;
+import org.apache.fop.accessibility.StructureTreeBuilder;
 import org.apache.fop.apps.FOUserAgent;
 import org.apache.fop.fo.ElementMapping;
 import org.apache.fop.fo.ElementMappingRegistry;
@@ -151,17 +151,17 @@
 
         private ContentHandler navParser;
 
-        private ParsedStructureTree structureTree;
+        private StructureTreeBuilder structureTreeBuilder;
 
-        private ContentHandler structureTreeBuilder;
+        private ContentHandler structureTreeBuilderWrapper;
 
-        private final class StructureTreeBuilder extends DelegatingContentHandler {
+        private final class StructureTreeBuilderWrapper extends DelegatingContentHandler {
 
             private Attributes pageSequenceAttributes;
 
-            private StructureTreeBuilder(Attributes pageSequenceAttributes,
-                    ParsedStructureTree structureTree) throws SAXException {
-                super(structureTree.getHandlerForNextPageSequence());
+            private StructureTreeBuilderWrapper(Attributes pageSequenceAttributes)
+                    throws SAXException {
+                super(structureTreeBuilder.getHandlerForNextPageSequence());
                 this.pageSequenceAttributes = new AttributesImpl(pageSequenceAttributes);
             }
 
@@ -196,8 +196,8 @@
             elementHandlers.put(EL_IMAGE, new ImageHandler());
 
             if (userAgent.isAccessibilityEnabled()) {
-                structureTree = new ParsedStructureTree(tFactory);
-                userAgent.setStructureTree(structureTree);
+                structureTreeBuilder = new StructureTreeBuilder(tFactory);
+                userAgent.setStructureTree(structureTreeBuilder.getStructureTree());
             }
         }
 
@@ -227,10 +227,10 @@
                 boolean handled = true;
                 if (NAMESPACE.equals(uri)) {
                     if (localName.equals(EL_PAGE_SEQUENCE) && userAgent.isAccessibilityEnabled()) {
-                        structureTreeBuilder = new StructureTreeBuilder(attributes, structureTree);
+                        structureTreeBuilderWrapper = new StructureTreeBuilderWrapper(attributes);
                     } else if (localName.equals(EL_STRUCTURE_TREE)) {
                         if (userAgent.isAccessibilityEnabled()) {
-                            delegate = structureTreeBuilder;
+                            delegate = structureTreeBuilderWrapper;
                         } else {
                             /* Delegate to a handler that does nothing */
                             delegate = new DefaultHandler();

Modified: xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/intermediate/IFSerializer.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/intermediate/IFSerializer.java?rev=825504&r1=825503&r2=825504&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/intermediate/IFSerializer.java (original)
+++ xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/intermediate/IFSerializer.java Thu Oct 15 14:25:28 2009
@@ -63,7 +63,7 @@
         implements IFConstants, IFPainter, IFDocumentNavigationHandler {
 
     private IFDocumentHandler mimicHandler;
-    private int pageSequenceCounter; // used for accessibility
+    private int pageSequenceIndex; // used for accessibility
 
     /** Holds the intermediate format state */
     private IFState state;
@@ -224,7 +224,7 @@
             if (this.getUserAgent().isAccessibilityEnabled()) {
                 StructureTree structureTree = getUserAgent().getStructureTree();
                 handler.startElement(EL_STRUCTURE_TREE); // add structure tree
-                NodeList nodes = structureTree.getPageSequence(++pageSequenceCounter);
+                NodeList nodes = structureTree.getPageSequence(pageSequenceIndex++);
                 for (int i = 0, n = nodes.getLength(); i < n; i++) {
                     Node node = nodes.item(i);
                     new DOM2SAX(handler).writeFragment(node);

Modified: xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/pdf/PDFDocumentHandler.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/pdf/PDFDocumentHandler.java?rev=825504&r1=825503&r2=825504&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/pdf/PDFDocumentHandler.java (original)
+++ xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/pdf/PDFDocumentHandler.java Thu Oct 15 14:25:28 2009
@@ -62,7 +62,7 @@
     /** logging instance */
     private static Log log = LogFactory.getLog(PDFDocumentHandler.class);
 
-    private int pageSequenceNumber;
+    private int pageSequenceIndex;
 
     private boolean accessEnabled;
 
@@ -186,8 +186,7 @@
         }
 
         if (accessEnabled) {
-            NodeList nodes = getUserAgent().getStructureTree().getPageSequence(
-                    ++pageSequenceNumber);
+            NodeList nodes = getUserAgent().getStructureTree().getPageSequence(pageSequenceIndex++);
             logicalStructureHandler.processStructureTree(nodes, getContext().getLanguage());
         }
     }

Modified: xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/pdf/PDFRenderer.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/pdf/PDFRenderer.java?rev=825504&r1=825503&r2=825504&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/pdf/PDFRenderer.java (original)
+++ xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/pdf/PDFRenderer.java Thu Oct 15 14:25:28 2009
@@ -204,7 +204,7 @@
 
     private PDFLogicalStructureHandler logicalStructureHandler;
 
-    private int pageSequenceNumber;
+    private int pageSequenceIndex;
 
     /** Reference in the structure tree to the image being rendered. */
     private String imageReference;
@@ -428,8 +428,7 @@
         }
         pdfUtil.generateDefaultXMPMetadata();
         if (accessEnabled) {
-            NodeList nodes = getUserAgent().getStructureTree().getPageSequence(
-                    ++pageSequenceNumber);
+            NodeList nodes = getUserAgent().getStructureTree().getPageSequence(pageSequenceIndex++);
             logicalStructureHandler.processStructureTree(nodes, language);
         }
     }

Modified: xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/xml/XMLRenderer.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/xml/XMLRenderer.java?rev=825504&r1=825503&r2=825504&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/xml/XMLRenderer.java (original)
+++ xmlgraphics/fop/branches/Temp_Accessibility/src/java/org/apache/fop/render/xml/XMLRenderer.java Thu Oct 15 14:25:28 2009
@@ -41,7 +41,6 @@
 import org.apache.xmlgraphics.util.QName;
 import org.apache.xmlgraphics.util.XMLizable;
 
-import org.apache.fop.accessibility.StructureTree;
 import org.apache.fop.apps.FOPException;
 import org.apache.fop.apps.FOUserAgent;
 import org.apache.fop.apps.MimeConstants;
@@ -108,7 +107,7 @@
     /** If not null, the XMLRenderer will mimic another renderer by using its font setup. */
     protected Renderer mimic;
 
-    private int pageSequenceNumber;
+    private int pageSequenceIndex;
 
     /**
      * Creates a new XML renderer.
@@ -446,10 +445,9 @@
         transferForeignObjects(pageSequence);
         startElement("pageSequence", atts);
         if (this.getUserAgent().isAccessibilityEnabled()) {
-            StructureTree structureTree = getUserAgent().getStructureTree();
             String structureTreeElement = "structureTree";
             startElement(structureTreeElement);
-            NodeList nodes = structureTree.getPageSequence(++pageSequenceNumber);
+            NodeList nodes = getUserAgent().getStructureTree().getPageSequence(pageSequenceIndex++);
             for (int i = 0, n = nodes.getLength(); i < n; i++) {
                 Node node = nodes.item(i);
                 try {



---------------------------------------------------------------------
To unsubscribe, e-mail: fop-commits-unsubscribe@xmlgraphics.apache.org
For additional commands, e-mail: fop-commits-help@xmlgraphics.apache.org