JDK-8145143 : Promote Image.impl_getUrl() to public API as Image.getUrl()
  • Type: Enhancement
  • Component: javafx
  • Sub-Component: scenegraph
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2015-12-10
  • Updated: 2016-03-29
  • Resolved: 2016-02-14
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 9
9Fixed
Related Reports
Relates :  
JDK-8136573 - Review (and possibly make public) candidate API from private APIs
Description
The Image.impl_getUrl() method is occasionally useful, and rather than remove it I think we should make it available as a public final method.

The one contentious point is that it is, again, a getter that is not a property method. See patch for suggested change.
Comments
Changeset: http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/3c95d7eccde7
14-02-2016
I am happy with the API as you have defined it. The only open question is whether to use Optional<String>; my take is that we should not for this class -- at least not without a good reason. Let me know what you think. The javadoc for the new getUrl method has a typo: * Returns the url used to use fetch the pixel data ... The "use" before "fetch" should be removed. Also, the "@return" tag is missing. +1 on the API (unless you want to discuss using Optional<String>)
11-02-2016
+1. The patch looks good to me.
10-12-2015