modules/graphics/src/test/java/test/javafx/scene/CSSNode.java

@@ -29,10 +29,11 @@
 import javafx.css.converter.SizeConverter;
 import com.sun.javafx.geom.BaseBounds;
 import com.sun.javafx.geom.transform.BaseTransform;
 import com.sun.javafx.jmx.MXNodeAlgorithm;
 import com.sun.javafx.jmx.MXNodeAlgorithmContext;
+import com.sun.javafx.scene.NodeHelper;
 import com.sun.javafx.sg.prism.NGGroup;
 import com.sun.javafx.sg.prism.NGNode;
 import javafx.beans.property.*;
 import javafx.css.*;
 import javafx.scene.paint.Color;

@@ -48,10 +49,26 @@
         CSSNodeHelper.setCSSNodeAccessor(new CSSNodeHelper.CSSNodeAccessor() {
             @Override
             public NGNode doCreatePeer(Node node) {
                 return ((CSSNode) node).doCreatePeer();
             }
+
+            @Override
+            public BaseBounds doComputeGeomBounds(Node node,
+            BaseBounds bounds, BaseTransform tx) {
+                return ((CSSNode) node).doComputeGeomBounds(bounds, tx);
+            }
+
+            @Override
+            public boolean doComputeContains(Node node, double localX, double localY) {
+                return ((CSSNode) node).doComputeContains(localX, localY);
+            }
+
+            @Override
+            public Object doProcessMXNode(Node node, MXNodeAlgorithm alg, MXNodeAlgorithmContext ctx) {
+                return ((CSSNode) node).doProcessMXNode(alg, ctx);
+            }
         });
     }
 
     {
         // To initialize the class helper at the begining each constructor of this class

@@ -150,11 +167,11 @@
         if (padding == null) {
             padding = new StyleableFloatProperty() {
 
                 @Override
                 protected void invalidated() {
-                    impl_geomChanged();
+                    NodeHelper.geomChanged(CSSNode.this);
                 }
 
                 @Override
                 public Object getBean() {
                     return CSSNode.this;

@@ -192,11 +209,11 @@
         if (contentSize == null) {
             contentSize = new SimpleFloatProperty() {
 
                 @Override
                 protected void invalidated() {
-                    impl_geomChanged();
+                    NodeHelper.geomChanged(CSSNode.this);
                 }
             };
         }
         return contentSize;
     }

@@ -234,21 +251,25 @@
 
     public boolean reapply = false;
     public boolean processCalled = false;
     public boolean applyCalled = false;
 
-    @Override
-    public BaseBounds impl_computeGeomBounds(BaseBounds bounds, BaseTransform tx) {
+    /*
+     * Note: This method MUST only be called via its accessor method.
+     */
+    private BaseBounds doComputeGeomBounds(BaseBounds bounds, BaseTransform tx) {
         if (bounds != null) {
             bounds = bounds.deriveWithNewBounds(0, 0, 0,
                     getContentSize() + getPadding() + getPadding(), getContentSize() + getPadding() + getPadding(), 0);
         }
         return bounds;
     }
 
-    @Override
-    protected boolean impl_computeContains(double localX, double localY) {
+    /*
+     * Note: This method MUST only be called via its accessor method.
+     */
+    private boolean doComputeContains(double localX, double localY) {
         // TODO: Missing code.
         return false;
     }
 
     private NGNode doCreatePeer() {

@@ -328,13 +349,12 @@
     @Override
     public List<CssMetaData<? extends Styleable, ?>> getCssMetaData() {
         return getClassCssMetaData();
     }
 
-    /**
-     * @treatAsPrivate Implementation detail
+    /*
+     * Note: This method MUST only be called via its accessor method.
      */
-    @Override
-    public Object impl_processMXNode(MXNodeAlgorithm alg, MXNodeAlgorithmContext ctx) {
+    private Object doProcessMXNode(MXNodeAlgorithm alg, MXNodeAlgorithmContext ctx) {
         return null;
     }
 }