modules/javafx.base/src/main/java/javafx/beans/property/adapter/ReadOnlyJavaBeanObjectProperty.java

rev 10441 : imported patch fix-8177566-trampoline
rev 10443 : [mq]: doc-8177566-trampoline

*** 1,7 ****
  /*
!  * Copyright (c) 2011, 2015, Oracle and/or its affiliates. All rights reserved.
   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
   *
   * This code is free software; you can redistribute it and/or modify it
   * under the terms of the GNU General Public License version 2 only, as
   * published by the Free Software Foundation.  Oracle designates this
--- 1,7 ----
  /*
!  * Copyright (c) 2011, 2017, Oracle and/or its affiliates. All rights reserved.
   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
   *
   * This code is free software; you can redistribute it and/or modify it
   * under the terms of the GNU General Public License version 2 only, as
   * published by the Free Software Foundation.  Oracle designates this
*** 23,32 ****
--- 23,33 ----
   * questions.
   */
  
  package javafx.beans.property.adapter;
  
+ import com.sun.javafx.property.MethodHelper;
  import com.sun.javafx.property.adapter.Disposer;
  import com.sun.javafx.property.adapter.ReadOnlyPropertyDescriptor;
  import javafx.beans.property.ReadOnlyObjectPropertyBase;
  
  import java.lang.reflect.InvocationTargetException;
*** 34,57 ****
  
  import java.security.AccessController;
  import java.security.AccessControlContext;
  import java.security.PrivilegedAction;
  
- import sun.reflect.misc.MethodUtil;
- 
  /**
   * A {@code ReadOnlyJavaBeanObjectProperty} provides an adapter between a regular
   * read only Java Bean property of {@code T} and a JavaFX
   * {@code ReadOnlyObjectProperty}. It cannot be created directly, but a
   * {@link ReadOnlyJavaBeanObjectPropertyBuilder} has to be used.
   * <p>
!  * As a minimum, the Java Bean must implement a getter for the
   * property. If the getter of an instance of this class is called, the property of
   * the Java Bean is returned. If the Java Bean property is bound (i.e. it supports
   * PropertyChangeListeners), this {@code ReadOnlyJavaBeanObjectProperty} will be
   * aware of changes in the Java Bean. Otherwise it can be notified about
   * changes by calling {@link #fireValueChangedEvent()}.
   *
   * @see javafx.beans.property.ReadOnlyObjectProperty
   * @see ReadOnlyJavaBeanObjectPropertyBuilder
   *
   * @param T the type of the wrapped {@code Object}
--- 35,64 ----
  
  import java.security.AccessController;
  import java.security.AccessControlContext;
  import java.security.PrivilegedAction;
  
  /**
   * A {@code ReadOnlyJavaBeanObjectProperty} provides an adapter between a regular
   * read only Java Bean property of {@code T} and a JavaFX
   * {@code ReadOnlyObjectProperty}. It cannot be created directly, but a
   * {@link ReadOnlyJavaBeanObjectPropertyBuilder} has to be used.
   * <p>
!  * As a minimum, the Java Bean class must implement a getter for the
   * property. If the getter of an instance of this class is called, the property of
   * the Java Bean is returned. If the Java Bean property is bound (i.e. it supports
   * PropertyChangeListeners), this {@code ReadOnlyJavaBeanObjectProperty} will be
   * aware of changes in the Java Bean. Otherwise it can be notified about
   * changes by calling {@link #fireValueChangedEvent()}.
+  * </p>
+  * <p>
+  * The Java Bean class must be declared public. If that class is in a named
+  * module, then the module must {@link Module#isOpen(String,Module) open}
+  * the containing package to at least the {@code javafx.base} module
+  * (or {@link Module#isExported(String) export} the containing package
+  * unconditionally).
+  * </p>
   *
   * @see javafx.beans.property.ReadOnlyObjectProperty
   * @see ReadOnlyJavaBeanObjectPropertyBuilder
   *
   * @param T the type of the wrapped {@code Object}
*** 80,90 ****
       */
      @Override
      public T get() {
          return AccessController.doPrivileged((PrivilegedAction<T>) () -> {
              try {
!                 return (T)MethodUtil.invoke(descriptor.getGetter(), getBean(), (Object[])null);
              } catch (IllegalAccessException e) {
                  throw new UndeclaredThrowableException(e);
              } catch (InvocationTargetException e) {
                  throw new UndeclaredThrowableException(e);
              }
--- 87,97 ----
       */
      @Override
      public T get() {
          return AccessController.doPrivileged((PrivilegedAction<T>) () -> {
              try {
!                 return (T)MethodHelper.invoke(descriptor.getGetter(), getBean(), (Object[])null);
              } catch (IllegalAccessException e) {
                  throw new UndeclaredThrowableException(e);
              } catch (InvocationTargetException e) {
                  throw new UndeclaredThrowableException(e);
              }