JDK-8090062 : Update java-wrappers for WebKit generated classes following WebKit update
  • Type: Bug
  • Component: javafx
  • Sub-Component: web
  • Affected Version: 9
  • Priority: P3
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2015-02-20
  • Updated: 2016-05-20
  • Resolved: 2015-07-31
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 8 JDK 9
8u72Fixed 9Fixed
Related Reports
Relates :  
Relates :  
Description
The com.sun.webkit.dom Java wrappers, which are generated by the native WebKit build, are also checked into modules/web/src/main/java-wrappers/com/sun/webkit/dom so that the Java code can build without requiring a build of the native WebKit.

These files need to be refreshed since the new merged webkit has changed the generated wrappers.
Comments
Ok, thanks. changeset: http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/0c0af0c28b43
31-07-2015

Seems not worth filing a new issue. So documenting here is fine with me.
31-07-2015

That is correct. I noticed that as well and made another webrev actually: http://cr.openjdk.java.net/~ant/JDK-8090062/webrev.1 but then discovered the class is not used. So, I left it excluded. I tried to understand why we generate it, but that wasn't easy from the first pass. So, we can file an issue or just have it journalized here...
31-07-2015

On Linux I see one file in the generated directory that isn't in java-wrappers: $ diff -r modules/web/build/linux/Release/WebCore/generated/java/com/sun/webkit/dom/ modules/web/src/main/java-wrappers/com/sun/webkit/dom/ Only in modules/web/build/linux/Release/WebCore/generated/java/com/sun/webkit/dom/: CSSUnknownRuleImpl.java As that file is unused there is no need to include it, however. +1
31-07-2015

Done. I've checked that it's build-able w/o -PCOMPILE_WEBKIT=true. webrev: http://cr.openjdk.java.net/~ant/JDK-8090062/webrev.0
29-07-2015