Index: src/java/org/apache/poi/hssf/dev/BiffDrawingToXml.java
===================================================================
--- src/java/org/apache/poi/hssf/dev/BiffDrawingToXml.java	(revision 1713221)
+++ src/java/org/apache/poi/hssf/dev/BiffDrawingToXml.java	(working copy)
@@ -24,14 +24,12 @@
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
-import java.lang.reflect.Field;
 import java.util.ArrayList;
 import java.util.List;
 
 import org.apache.poi.ddf.EscherRecord;
 import org.apache.poi.hssf.model.InternalWorkbook;
 import org.apache.poi.hssf.record.DrawingGroupRecord;
-import org.apache.poi.hssf.record.EscherAggregate;
 import org.apache.poi.hssf.usermodel.HSSFPatriarch;
 import org.apache.poi.hssf.usermodel.HSSFWorkbook;
 import org.apache.poi.poifs.filesystem.NPOIFSFileSystem;
@@ -137,7 +135,7 @@
     public static void writeToFile(OutputStream fos, InputStream xlsWorkbook, boolean excludeWorkbookRecords, String[] params) throws IOException {
         NPOIFSFileSystem fs = new NPOIFSFileSystem(xlsWorkbook);
         HSSFWorkbook workbook = new HSSFWorkbook(fs);
-        InternalWorkbook internalWorkbook = getInternalWorkbook(workbook);
+        InternalWorkbook internalWorkbook = workbook.getInternalWorkbook();
         DrawingGroupRecord r = (DrawingGroupRecord) internalWorkbook.findFirstRecordBySid(DrawingGroupRecord.sid);
         r.decode();
 
@@ -155,7 +153,7 @@
             HSSFPatriarch p = workbook.getSheetAt(i).getDrawingPatriarch();
             if(p != null ) {
                 builder.append(tab).append("<sheet").append(i).append(">\n");
-                builder.append(getHSSFPatriarchBoundAggregate(p).toXml(tab + "\t"));
+                builder.append(p.getBoundAggregate().toXml(tab + "\t"));
                 builder.append(tab).append("</sheet").append(i).append(">\n");
             }
         }
@@ -164,31 +162,4 @@
         fos.close();
     }
 
-    private static EscherAggregate getHSSFPatriarchBoundAggregate(HSSFPatriarch patriarch) {
-        Field boundAggregateField = null;
-        try {
-            boundAggregateField = patriarch.getClass().getDeclaredField("_boundAggregate");
-            boundAggregateField.setAccessible(true);
-            return (EscherAggregate) boundAggregateField.get(patriarch);
-        } catch (NoSuchFieldException e) {
-            e.printStackTrace();
-        } catch (IllegalAccessException e) {
-            e.printStackTrace();
-        }
-        return null;
-    }
-
-    private static InternalWorkbook getInternalWorkbook(HSSFWorkbook workbook) {
-        Field internalSheetField = null;
-        try {
-            internalSheetField = workbook.getClass().getDeclaredField("workbook");
-            internalSheetField.setAccessible(true);
-            return (InternalWorkbook) internalSheetField.get(workbook);
-        } catch (NoSuchFieldException e) {
-            e.printStackTrace();
-        } catch (IllegalAccessException e) {
-            e.printStackTrace();
-        }
-        return null;
-    }
 }
Index: src/java/org/apache/poi/hssf/usermodel/HSSFComment.java
===================================================================
--- src/java/org/apache/poi/hssf/usermodel/HSSFComment.java	(revision 1713221)
+++ src/java/org/apache/poi/hssf/usermodel/HSSFComment.java	(working copy)
@@ -87,7 +87,7 @@
     @Override
     void afterInsert(HSSFPatriarch patriarch) {
         super.afterInsert(patriarch);
-        patriarch._getBoundAggregate().addTailRecord(getNoteRecord());
+        patriarch.getBoundAggregate().addTailRecord(getNoteRecord());
     }
 
     @Override
@@ -253,7 +253,7 @@
 
     public void afterRemove(HSSFPatriarch patriarch){
         super.afterRemove(patriarch);
-        patriarch._getBoundAggregate().removeTailRecord(getNoteRecord());
+        patriarch.getBoundAggregate().removeTailRecord(getNoteRecord());
     }
 
     @Override
Index: src/java/org/apache/poi/hssf/usermodel/HSSFObjectData.java
===================================================================
--- src/java/org/apache/poi/hssf/usermodel/HSSFObjectData.java	(revision 1713221)
+++ src/java/org/apache/poi/hssf/usermodel/HSSFObjectData.java	(working copy)
@@ -126,7 +126,7 @@
 
     @Override
     void afterInsert(HSSFPatriarch patriarch) {
-        EscherAggregate agg = patriarch._getBoundAggregate();
+        EscherAggregate agg = patriarch.getBoundAggregate();
         agg.associateShapeToObjRecord(getEscherContainer().getChildById(EscherClientDataRecord.RECORD_ID), getObjRecord());
         EscherBSERecord bse =
                 patriarch.getSheet().getWorkbook().getWorkbook().getBSERecord(getPictureIndex());
Index: src/java/org/apache/poi/hssf/usermodel/HSSFPatriarch.java
===================================================================
--- src/java/org/apache/poi/hssf/usermodel/HSSFPatriarch.java	(revision 1713221)
+++ src/java/org/apache/poi/hssf/usermodel/HSSFPatriarch.java	(working copy)
@@ -495,7 +495,8 @@
      * Returns the aggregate escher record we're bound to
      * @return - low level representation of sheet drawing data
      */
-    protected EscherAggregate _getBoundAggregate() {
+    @Internal
+    public EscherAggregate getBoundAggregate() {
         return _boundAggregate;
     }
 
Index: src/java/org/apache/poi/hssf/usermodel/HSSFPicture.java
===================================================================
--- src/java/org/apache/poi/hssf/usermodel/HSSFPicture.java	(revision 1713221)
+++ src/java/org/apache/poi/hssf/usermodel/HSSFPicture.java	(working copy)
@@ -211,7 +211,7 @@
 
     @Override
     void afterInsert(HSSFPatriarch patriarch) {
-        EscherAggregate agg = patriarch._getBoundAggregate();
+        EscherAggregate agg = patriarch.getBoundAggregate();
         agg.associateShapeToObjRecord(getEscherContainer().getChildById(EscherClientDataRecord.RECORD_ID), getObjRecord());
         EscherBSERecord bse =
                 patriarch.getSheet().getWorkbook().getWorkbook().getBSERecord(getPictureIndex());
Index: src/java/org/apache/poi/hssf/usermodel/HSSFPolygon.java
===================================================================
--- src/java/org/apache/poi/hssf/usermodel/HSSFPolygon.java	(revision 1713221)
+++ src/java/org/apache/poi/hssf/usermodel/HSSFPolygon.java	(working copy)
@@ -116,7 +116,7 @@
 
     @Override
     protected void afterRemove(HSSFPatriarch patriarch) {
-        patriarch._getBoundAggregate().removeShapeToObjRecord(getEscherContainer().getChildById(EscherClientDataRecord.RECORD_ID));
+        patriarch.getBoundAggregate().removeShapeToObjRecord(getEscherContainer().getChildById(EscherClientDataRecord.RECORD_ID));
     }
 
     /**
Index: src/java/org/apache/poi/hssf/usermodel/HSSFShapeGroup.java
===================================================================
--- src/java/org/apache/poi/hssf/usermodel/HSSFShapeGroup.java	(revision 1713221)
+++ src/java/org/apache/poi/hssf/usermodel/HSSFShapeGroup.java	(working copy)
@@ -122,7 +122,7 @@
 
     @Override
     protected void afterRemove(HSSFPatriarch patriarch) {
-        patriarch._getBoundAggregate().removeShapeToObjRecord(getEscherContainer().getChildContainers().get(0)
+        patriarch.getBoundAggregate().removeShapeToObjRecord(getEscherContainer().getChildContainers().get(0)
                 .getChildById(EscherClientDataRecord.RECORD_ID));
         for ( int i=0; i<shapes.size(); i++ ) {
             HSSFShape shape = shapes.get(i);
@@ -314,7 +314,7 @@
 
     @Override
     void afterInsert(HSSFPatriarch patriarch){
-        EscherAggregate agg = patriarch._getBoundAggregate();
+        EscherAggregate agg = patriarch.getBoundAggregate();
         EscherContainerRecord containerRecord = getEscherContainer().getChildById(EscherContainerRecord.SP_CONTAINER);
         agg.associateShapeToObjRecord(containerRecord.getChildById(EscherClientDataRecord.RECORD_ID), getObjRecord());
     }
Index: src/java/org/apache/poi/hssf/usermodel/HSSFSheet.java
===================================================================
--- src/java/org/apache/poi/hssf/usermodel/HSSFSheet.java	(revision 1713221)
+++ src/java/org/apache/poi/hssf/usermodel/HSSFSheet.java	(working copy)
@@ -148,7 +148,7 @@
         }
         if (getDrawingPatriarch() != null) {
             HSSFPatriarch patr = HSSFPatriarch.createPatriarch(this.getDrawingPatriarch(), sheet);
-            sheet._sheet.getRecords().add(pos, patr._getBoundAggregate());
+            sheet._sheet.getRecords().add(pos, patr.getBoundAggregate());
             sheet._patriarch = patr;
         }
         return sheet;
Index: src/java/org/apache/poi/hssf/usermodel/HSSFSimpleShape.java
===================================================================
--- src/java/org/apache/poi/hssf/usermodel/HSSFSimpleShape.java	(revision 1713221)
+++ src/java/org/apache/poi/hssf/usermodel/HSSFSimpleShape.java	(working copy)
@@ -144,9 +144,9 @@
 
     @Override
     protected void afterRemove(HSSFPatriarch patriarch) {
-        patriarch._getBoundAggregate().removeShapeToObjRecord(getEscherContainer().getChildById(EscherClientDataRecord.RECORD_ID));
+        patriarch.getBoundAggregate().removeShapeToObjRecord(getEscherContainer().getChildById(EscherClientDataRecord.RECORD_ID));
         if (null != getEscherContainer().getChildById(EscherTextboxRecord.RECORD_ID)){
-            patriarch._getBoundAggregate().removeShapeToObjRecord(getEscherContainer().getChildById(EscherTextboxRecord.RECORD_ID));
+            patriarch.getBoundAggregate().removeShapeToObjRecord(getEscherContainer().getChildById(EscherTextboxRecord.RECORD_ID));
         }
     }
 
@@ -177,7 +177,7 @@
 
     @Override
     void afterInsert(HSSFPatriarch patriarch){
-        EscherAggregate agg = patriarch._getBoundAggregate();
+        EscherAggregate agg = patriarch.getBoundAggregate();
         agg.associateShapeToObjRecord(getEscherContainer().getChildById(EscherClientDataRecord.RECORD_ID), getObjRecord());
 
         if (null != getTextObjectRecord()){
@@ -244,7 +244,7 @@
             escherTextbox.setRecordId(EscherTextboxRecord.RECORD_ID);
             escherTextbox.setOptions((short) 0x0000);
             getEscherContainer().addChildRecord(escherTextbox);
-            getPatriarch()._getBoundAggregate().associateShapeToObjRecord(escherTextbox, _textObjectRecord);
+            getPatriarch().getBoundAggregate().associateShapeToObjRecord(escherTextbox, _textObjectRecord);
         }
         return _textObjectRecord;
     }
Index: src/java/org/apache/poi/hssf/usermodel/HSSFTextbox.java
===================================================================
--- src/java/org/apache/poi/hssf/usermodel/HSSFTextbox.java	(revision 1713221)
+++ src/java/org/apache/poi/hssf/usermodel/HSSFTextbox.java	(working copy)
@@ -128,7 +128,7 @@
 
     @Override
     void afterInsert(HSSFPatriarch patriarch) {
-        EscherAggregate agg = patriarch._getBoundAggregate();
+        EscherAggregate agg = patriarch.getBoundAggregate();
         agg.associateShapeToObjRecord(getEscherContainer().getChildById(EscherClientDataRecord.RECORD_ID), getObjRecord());
         if (getTextObjectRecord() != null){
             agg.associateShapeToObjRecord(getEscherContainer().getChildById(EscherTextboxRecord.RECORD_ID), getTextObjectRecord());
@@ -240,7 +240,7 @@
 
     @Override
     protected void afterRemove(HSSFPatriarch patriarch) {
-        patriarch._getBoundAggregate().removeShapeToObjRecord(getEscherContainer().getChildById(EscherClientDataRecord.RECORD_ID));
-        patriarch._getBoundAggregate().removeShapeToObjRecord(getEscherContainer().getChildById(EscherTextboxRecord.RECORD_ID));
+        patriarch.getBoundAggregate().removeShapeToObjRecord(getEscherContainer().getChildById(EscherClientDataRecord.RECORD_ID));
+        patriarch.getBoundAggregate().removeShapeToObjRecord(getEscherContainer().getChildById(EscherTextboxRecord.RECORD_ID));
     }
 }
Index: src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java
===================================================================
--- src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java	(revision 1713221)
+++ src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java	(working copy)
@@ -89,6 +89,7 @@
 import org.apache.poi.ss.util.WorkbookUtil;
 import org.apache.poi.util.Configurator;
 import org.apache.poi.util.HexDump;
+import org.apache.poi.util.Internal;
 import org.apache.poi.util.LittleEndian;
 import org.apache.poi.util.POILogFactory;
 import org.apache.poi.util.POILogger;
@@ -2147,4 +2148,9 @@
     public DirectoryNode getRootDirectory(){
         return directory;
     }
+    
+    @Internal
+    public InternalWorkbook getInternalWorkbook() {
+        return workbook;
+    }
 }
Index: src/java/org/apache/poi/poifs/dev/POIFSDump.java
===================================================================
--- src/java/org/apache/poi/poifs/dev/POIFSDump.java	(revision 1713221)
+++ src/java/org/apache/poi/poifs/dev/POIFSDump.java	(working copy)
@@ -21,7 +21,6 @@
 import java.io.FileOutputStream;
 import java.io.IOException;
 import java.io.OutputStream;
-import java.lang.reflect.Field;
 import java.nio.ByteBuffer;
 import java.util.Iterator;
 
@@ -73,11 +72,11 @@
             dump(root, file);
             
             if (dumpProps) {
-                HeaderBlock header = getHeaderBlock(fs);
+                HeaderBlock header = fs.getHeaderBlock();
                 dump(fs, header.getPropertyStart(), "properties", file);
             }
             if (dumpMini) {
-                NPropertyTable props = getPropertyTable(fs);
+                NPropertyTable props = fs.getPropertyTable();
                 int startBlock = props.getRoot().getStartBlock(); 
                 if (startBlock == POIFSConstants.END_OF_CHAIN) {
                     System.err.println("No Mini Stream in file");
@@ -86,21 +85,8 @@
                 }
             }
         }
-   }
+    }
     
-    protected static HeaderBlock getHeaderBlock(NPOIFSFileSystem fs) throws Exception {
-        Field headerF = NPOIFSFileSystem.class.getDeclaredField("_header");
-        headerF.setAccessible(true);
-        HeaderBlock header = (HeaderBlock)headerF.get(fs);
-        return header;
-    }
-    protected static NPropertyTable getPropertyTable(NPOIFSFileSystem fs) throws Exception {
-        Field ptF = NPOIFSFileSystem.class.getDeclaredField("_property_table");
-        ptF.setAccessible(true);
-        NPropertyTable table = (NPropertyTable)ptF.get(fs);
-        return table;
-    }
-
     public static void dump(DirectoryEntry root, File parent) throws IOException {
         for(Iterator<Entry> it = root.getEntries(); it.hasNext();){
             Entry entry = it.next();
Index: src/java/org/apache/poi/poifs/dev/POIFSHeaderDumper.java
===================================================================
--- src/java/org/apache/poi/poifs/dev/POIFSHeaderDumper.java	(revision 1713221)
+++ src/java/org/apache/poi/poifs/dev/POIFSHeaderDumper.java	(working copy)
@@ -19,8 +19,6 @@
 
 import java.io.FileInputStream;
 import java.io.InputStream;
-import java.lang.reflect.Field;
-import java.lang.reflect.Method;
 
 import org.apache.poi.poifs.common.POIFSBigBlockSize;
 import org.apache.poi.poifs.common.POIFSConstants;
@@ -118,11 +116,8 @@
         System.out.println("Raw Blocks Details:");
         System.out.println(" Number of blocks: " + data_blocks.blockCount());
 
-        Method gbm = data_blocks.getClass().getSuperclass().getDeclaredMethod("get", int.class);
-        gbm.setAccessible(true);
-
         for(int i=0; i<Math.min(16, data_blocks.blockCount()); i++) {
-            ListManagedBlock block = (ListManagedBlock)gbm.invoke(data_blocks, Integer.valueOf(i));
+            ListManagedBlock block = data_blocks.get(i);
             byte[] data = new byte[Math.min(48, block.getData().length)];
             System.arraycopy(block.getData(), 0, data, 0, data.length);
 
@@ -135,9 +130,7 @@
 
     public static void displayBATReader(String type, BlockAllocationTableReader batReader) throws Exception {
         System.out.println("Sectors, as referenced from the "+type+" FAT:");
-        Field entriesF = batReader.getClass().getDeclaredField("_entries");
-        entriesF.setAccessible(true);
-        IntList entries = (IntList)entriesF.get(batReader);
+        IntList entries = batReader.getEntries();
 
         for(int i=0; i<entries.size(); i++) {
             int bn = entries.get(i);
Index: src/java/org/apache/poi/poifs/filesystem/NPOIFSFileSystem.java
===================================================================
--- src/java/org/apache/poi/poifs/filesystem/NPOIFSFileSystem.java	(revision 1713221)
+++ src/java/org/apache/poi/poifs/filesystem/NPOIFSFileSystem.java	(working copy)
@@ -55,6 +55,7 @@
 import org.apache.poi.poifs.storage.HeaderBlockWriter;
 import org.apache.poi.util.CloseIgnoringInputStream;
 import org.apache.poi.util.IOUtils;
+import org.apache.poi.util.Internal;
 import org.apache.poi.util.LongField;
 
 /**
@@ -970,5 +971,15 @@
     protected int getBlockStoreBlockSize() {
        return getBigBlockSize();
     }
+
+    @Internal
+    public NPropertyTable getPropertyTable() {
+        return _property_table;
+    }
+
+    @Internal
+    public HeaderBlock getHeaderBlock() {
+        return _header;
+    }
 }
 
Index: src/java/org/apache/poi/poifs/storage/BlockAllocationTableReader.java
===================================================================
--- src/java/org/apache/poi/poifs/storage/BlockAllocationTableReader.java	(revision 1713221)
+++ src/java/org/apache/poi/poifs/storage/BlockAllocationTableReader.java	(working copy)
@@ -312,4 +312,9 @@
         }
         raw_blocks.setBAT(this);
     }
+
+    @Internal
+    public IntList getEntries() {
+        return _entries;
+    }
 }
Index: src/java/org/apache/poi/poifs/storage/BlockListImpl.java
===================================================================
--- src/java/org/apache/poi/poifs/storage/BlockListImpl.java	(revision 1713221)
+++ src/java/org/apache/poi/poifs/storage/BlockListImpl.java	(working copy)
@@ -19,6 +19,8 @@
 
 import java.io.IOException;
 
+import org.apache.poi.util.Internal;
+
 /**
  * A simple implementation of BlockList
  *
@@ -59,10 +61,11 @@
     }
 
     /**
-     * Unit testing method. Gets, without sanity checks or
+     * Internal method. Gets, without sanity checks or
      *  removing.
      */
-    protected ListManagedBlock get(final int index) {
+    @Internal
+    public ListManagedBlock get(final int index) {
         return _blocks[index];
     }
 
Index: src/ooxml/java/org/apache/poi/POIXMLDocumentPartAccessor.java
===================================================================
--- src/ooxml/java/org/apache/poi/POIXMLDocumentPartAccessor.java	(revision 0)
+++ src/ooxml/java/org/apache/poi/POIXMLDocumentPartAccessor.java	(working copy)
@@ -0,0 +1,35 @@
+/* ====================================================================
+   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.
+==================================================================== */
+package org.apache.poi;
+
+import java.io.IOException;
+
+import org.apache.poi.util.Internal;
+
+/**
+ * Internal class, do not use
+ */
+@Internal
+public final class POIXMLDocumentPartAccessor {
+
+    private POIXMLDocumentPartAccessor() {}
+    
+    @Internal
+    public static void invokeOnDocumentRead(POIXMLDocumentPart part) throws IOException {
+        part.onDocumentRead();
+    }
+}
Index: src/ooxml/java/org/apache/poi/POIXMLDocumentPartAccessor.java
===================================================================
--- src/ooxml/java/org/apache/poi/POIXMLDocumentPartAccessor.java	(revision 0)
+++ src/ooxml/java/org/apache/poi/POIXMLDocumentPartAccessor.java	(working copy)

Property changes on: src/ooxml/java/org/apache/poi/POIXMLDocumentPartAccessor.java
___________________________________________________________________
Added: svn:eol-style
## -0,0 +1 ##
+native
\ No newline at end of property
Index: src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFDocument.java
===================================================================
--- src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFDocument.java	(revision 1713221)
+++ src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFDocument.java	(working copy)
@@ -20,7 +20,6 @@
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
-import java.lang.reflect.Method;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
@@ -35,6 +34,7 @@
 
 import org.apache.poi.POIXMLDocument;
 import org.apache.poi.POIXMLDocumentPart;
+import org.apache.poi.POIXMLDocumentPartAccessor;
 import org.apache.poi.POIXMLException;
 import org.apache.poi.POIXMLProperties;
 import org.apache.poi.POIXMLRelation;
@@ -48,7 +48,6 @@
 import org.apache.poi.openxml4j.opc.PackagingURIHelper;
 import org.apache.poi.openxml4j.opc.TargetMode;
 import org.apache.poi.poifs.crypt.HashAlgorithm;
-import org.apache.poi.util.DocumentHelper;
 import org.apache.poi.util.IOUtils;
 import org.apache.poi.util.IdentifierManager;
 import org.apache.poi.util.Internal;
@@ -153,6 +152,7 @@
         }
     }
 
+    @SuppressWarnings("deprecation")
     @Override
     protected void onDocumentRead() throws IOException {
         try {
@@ -224,13 +224,7 @@
                     for (POIXMLDocumentPart gp : p.getRelations()) {
                         // Trigger the onDocumentRead for all the child parts
                         // Otherwise we'll hit issues on Styles, Settings etc on save
-                        try {
-                            Method onDocumentRead = gp.getClass().getDeclaredMethod("onDocumentRead");
-                            onDocumentRead.setAccessible(true);
-                            onDocumentRead.invoke(gp);
-                        } catch (Exception e) {
-                            throw new POIXMLException(e);
-                        }
+                        POIXMLDocumentPartAccessor.invokeOnDocumentRead(gp);
                     }
                 }
             }
Index: src/testcases/org/apache/poi/hssf/usermodel/HSSFTestHelper.java
===================================================================
--- src/testcases/org/apache/poi/hssf/usermodel/HSSFTestHelper.java	(revision 1713221)
+++ src/testcases/org/apache/poi/hssf/usermodel/HSSFTestHelper.java	(working copy)
@@ -72,7 +72,7 @@
     }
 
     public static EscherAggregate getEscherAggregate(HSSFPatriarch patriarch){
-        return patriarch._getBoundAggregate();
+        return patriarch.getBoundAggregate();
     }
 
     public static int allocateNewShapeId(HSSFPatriarch patriarch){
Index: src/testcases/org/apache/poi/hssf/usermodel/TestCloneSheet.java
===================================================================
--- src/testcases/org/apache/poi/hssf/usermodel/TestCloneSheet.java	(revision 1713221)
+++ src/testcases/org/apache/poi/hssf/usermodel/TestCloneSheet.java	(working copy)
@@ -85,13 +85,13 @@
         HSSFSheet s = b.createSheet("Test");
         HSSFPatriarch patriarch = s.createDrawingPatriarch();
 
-        EscherAggregate agg1 = patriarch._getBoundAggregate();
+        EscherAggregate agg1 = patriarch.getBoundAggregate();
 
         HSSFSheet s2 = s.cloneSheet(b);
 
         patriarch = s2.getDrawingPatriarch();
 
-        EscherAggregate agg2 = patriarch._getBoundAggregate();
+        EscherAggregate agg2 = patriarch.getBoundAggregate();
 
         EscherSpRecord sp1 = (EscherSpRecord) agg1.getEscherContainer().getChild(1).getChild(0).getChild(1);
         EscherSpRecord sp2 = (EscherSpRecord) agg2.getEscherContainer().getChild(1).getChild(0).getChild(1);
Index: src/testcases/org/apache/poi/hssf/usermodel/TestHSSFSheet.java
===================================================================
--- src/testcases/org/apache/poi/hssf/usermodel/TestHSSFSheet.java	(revision 1713221)
+++ src/testcases/org/apache/poi/hssf/usermodel/TestHSSFSheet.java	(working copy)
@@ -940,8 +940,8 @@
         HSSFSheet sheet2 = wb2.getSheetAt(1);
 
         //check that id of the drawing group was updated
-        EscherDgRecord dg1 = (EscherDgRecord)sheet1.getDrawingPatriarch()._getBoundAggregate().findFirstWithId(EscherDgRecord.RECORD_ID);
-        EscherDgRecord dg2 = (EscherDgRecord)sheet2.getDrawingPatriarch()._getBoundAggregate().findFirstWithId(EscherDgRecord.RECORD_ID);
+        EscherDgRecord dg1 = (EscherDgRecord)sheet1.getDrawingPatriarch().getBoundAggregate().findFirstWithId(EscherDgRecord.RECORD_ID);
+        EscherDgRecord dg2 = (EscherDgRecord)sheet2.getDrawingPatriarch().getBoundAggregate().findFirstWithId(EscherDgRecord.RECORD_ID);
         int dg_id_1 = dg1.getOptions() >> 4;
         int dg_id_2 = dg2.getOptions() >> 4;
         assertEquals(dg_id_1 + 1, dg_id_2);
Index: src/testcases/org/apache/poi/hssf/usermodel/TestPatriarch.java
===================================================================
--- src/testcases/org/apache/poi/hssf/usermodel/TestPatriarch.java	(revision 1713221)
+++ src/testcases/org/apache/poi/hssf/usermodel/TestPatriarch.java	(working copy)
@@ -40,7 +40,7 @@
 
         assertSame(patriarch, sh.getDrawingPatriarch());
 
-        EscherAggregate agg = patriarch._getBoundAggregate();
+        EscherAggregate agg = patriarch.getBoundAggregate();
 
         EscherDgRecord dg = agg.getEscherContainer().getChildById(EscherDgRecord.RECORD_ID);
         int lastId = dg.getLastMSOSPID();
@@ -50,7 +50,7 @@
         wb = HSSFTestDataSamples.writeOutAndReadBack(wb);
         sh = wb.getSheetAt(0);
         patriarch = sh.createDrawingPatriarch();
-        dg = patriarch._getBoundAggregate().getEscherContainer().getChildById(EscherDgRecord.RECORD_ID);
+        dg = patriarch.getBoundAggregate().getEscherContainer().getChildById(EscherDgRecord.RECORD_ID);
 
         assertEquals(lastId, dg.getLastMSOSPID());
     }