8172095: Let Node.managed become CSS-styleable

Reviewed-by: kcr, arapte

Author: abhinayagarwal

modules/javafx.graphics/src/main/docs/javafx/scene/doc-files/cssref.html

@@ -1750,6 +1750,13 @@ <h4><a id="node">Node</a></h4>
           <td>See <a href="http://www.w3.org/TR/CSS2/visufx.html#visibility">W3C
               visibility property</a></td>
         </tr>
+        <tr>
+            <th class="propertyname" scope="row">-fx-managed</th>
+            <td class="value"><a href="#typeboolean" class="typelink">&lt;boolean&gt;</a></td>
+            <td class="default">true</td>
+            <td class="range">&nbsp;</td>
+            <td>Defines whether this node's layout will be managed by its parent</td>
+        </tr>
       </tbody>
     </table>
     <h4>Pseudo-classes</h4>

modules/javafx.graphics/src/main/java/javafx/scene/Node.java

@@ -2637,12 +2637,12 @@ public final void setManaged(boolean value) {
     }
 
     public final boolean isManaged() {
-        return managed == null ? true : managed.get();
+        return managed == null || managed.get();
     }
 
     public final BooleanProperty managedProperty() {
         if (managed == null) {
-            managed = new BooleanPropertyBase(true) {
+            managed = new StyleableBooleanProperty(true) {
 
                 @Override
                 protected void invalidated() {
@@ -2653,6 +2653,11 @@ protected void invalidated() {
                     notifyManagedChanged();
                 }
 
+                @Override
+                public CssMetaData<Node, Boolean> getCssMetaData() {
+                    return StyleableProperties.MANAGED;
+                }
+
                 @Override
                 public Object getBean() {
                     return Node.this;
@@ -9101,6 +9106,20 @@ public StyleableProperty<Boolean> getStyleableProperty(Node node) {
                     return (StyleableProperty<Boolean>)node.visibleProperty();
                 }
             };
+        private static final CssMetaData<Node,Boolean> MANAGED =
+            new CssMetaData<Node,Boolean>("-fx-managed",
+                    BooleanConverter.getInstance(), Boolean.TRUE) {
+
+                @Override
+                public boolean isSettable(Node node) {
+                    return node.managed == null || !node.managed.isBound();
+                }
+
+                @Override
+                public StyleableProperty<Boolean> getStyleableProperty(Node node) {
+                    return (StyleableProperty<Boolean>)node.managedProperty();
+                }
+            };
 
          private static final List<CssMetaData<? extends Styleable, ?>> STYLEABLES;
 
@@ -9122,6 +9141,7 @@ public StyleableProperty<Boolean> getStyleableProperty(Node node) {
              styleables.add(TRANSLATE_Y);
              styleables.add(TRANSLATE_Z);
              styleables.add(VISIBILITY);
+             styleables.add(MANAGED);
              STYLEABLES = Collections.unmodifiableList(styleables);
 
          }

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

@@ -1090,6 +1090,34 @@ public void settingVisibleThroughPropertyShouldAffectBothGetterAndProperty() {
         assertFalse(node.visibleProperty().get());
     }
 
+    @Test
+    public void testDefaultValueForManagedIsTrueWhenReadFromGetter() {
+        final Node node = new Rectangle();
+        assertTrue(node.isManaged());
+    }
+
+    @Test
+    public void testDefaultValueForManagedIsTrueWhenReadFromProperty() {
+        final Node node = new Rectangle();
+        assertTrue(node.managedProperty().get());
+    }
+
+    @Test
+    public void settingManagedThroughSetterShouldAffectBothGetterAndProperty() {
+        final Node node = new Rectangle();
+        node.setManaged(false);
+        assertFalse(node.isManaged());
+        assertFalse(node.managedProperty().get());
+    }
+
+    @Test
+    public void settingManagedThroughPropertyShouldAffectBothGetterAndProperty() {
+        final Node node = new Rectangle();
+        node.managedProperty().set(false);
+        assertFalse(node.isManaged());
+        assertFalse(node.managedProperty().get());
+    }
+
     @Test
     public void testDefaultStyleIsEmptyString() {
         final Node node = new Rectangle();
@@ -1894,6 +1922,14 @@ public void rtlSceneSizeShouldBeComputedCorrectly() {
         assertEquals(100.0, scene.getWidth(), 0.00001);
     }
 
+    @Test public void managedSetFromCSS() {
+        final AnchorPane node = new AnchorPane();
+        node.setStyle("-fx-managed: false");
+        Scene s = new Scene(node);
+        node.applyCss();
+        assertFalse(node.isManaged());
+    }
+
     private Node createTestRect() {
         final Rectangle rect = new StubRect();
         Scene scene = new Scene(new Group(rect));

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

@@ -87,8 +87,8 @@ public boolean equals(final Object expected,
                    }),
             config("cursor", null, "-fx-cursor", Cursor.MOVE),
             config("effect", null, "-fx-effect", new Shadow()),
-            config("focusTraversable", false,
-                   "-fx-focus-traversable", true),
+            config("focusTraversable", false, "-fx-focus-traversable", true),
+            config("managed", true, "-fx-managed", false, false),
             config("opacity", 1.0, "-fx-opacity", 0.5),
             config("opacity", 0.5, "-fx-opacity", null, 0.0),
             config("viewOrder", 0.0, "-fx-view-order", 0.5),