Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8138652

[macosx] New WebView Native Code uses private Apple APIs

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: P3
    • Resolution: Fixed
    • Affects Version/s: 8u60, 9
    • Fix Version/s: 9
    • Component/s: javafx
    • Subcomponent:
      web
    • OS:
      os_x

      Backports

        Description

        The new update to the QTKit code for JavaFX Web View uses some APIs that apple considers "private."

            ubrk_getRuleStatus
            ubrk_setUText
            ucnv_getCanonicalName
            ucnv_reset
            ucol_strcollIter

        If you do some web searching you will find some QTWebKit bugs and blog posts that provide instructions for a Mac App Store "clean" way to build it, so it can be fixed.

        Right now packager has to remove the dylib, much in the same way the media libraries were excluded for QuickTime APIs.

          Issue Links

            Activity

            Hide
            scfitch Stephen Fitch added a comment -
            Issue blocks app submission to Apple (Mac) app store.
            Show
            scfitch Stephen Fitch added a comment - Issue blocks app submission to Apple (Mac) app store.
            Hide
            cbensen Chris Bensen added a comment -
            If this is fixed in 8u then please file a bug against the Java Packager to not strip out libjfxwebkit.dylib
            Show
            cbensen Chris Bensen added a comment - If this is fixed in 8u then please file a bug against the Java Packager to not strip out libjfxwebkit.dylib
            Hide
            ghb Guru Hb added a comment -
            webrev : http://cr.openjdk.java.net/~ghb/8138652/webrev.00/
            Root cause : We were dynamically linked native library "libicucore.dylib" provided by OS X framework. Some of the API (mentioned in Description) are public. This restricts our javafx bundled with native webkit library getting rejected from app store.

            Solution: Use statically linked lib icu (which we are doing for windows and linux) with native webkit.
            Show
            ghb Guru Hb added a comment - webrev : http://cr.openjdk.java.net/~ghb/8138652/webrev.00/ Root cause : We were dynamically linked native library "libicucore.dylib" provided by OS X framework. Some of the API (mentioned in Description) are public. This restricts our javafx bundled with native webkit library getting rejected from app store. Solution: Use statically linked lib icu (which we are doing for windows and linux) with native webkit.
            Hide
            arajkumar Arunprasad Rajkumar added a comment -
            lgtm
            Show
            arajkumar Arunprasad Rajkumar added a comment - lgtm
            Hide
            mbilla Murali Billa added a comment -
            +1
            Show
            mbilla Murali Billa added a comment - +1
            Hide
            kcr Kevin Rushforth added a comment -
            I get build failures with your patch applied. These will need to be resolved before we can proceed. I am compiling on Mac OS X 10.9.5 which matches our production build machines.
            Show
            kcr Kevin Rushforth added a comment - I get build failures with your patch applied. These will need to be resolved before we can proceed. I am compiling on Mac OS X 10.9.5 which matches our production build machines.
            Hide
            kcr Kevin Rushforth added a comment -
            Build failure is resolved.

            +1
            Show
            kcr Kevin Rushforth added a comment - Build failure is resolved. +1
            Hide
            ghb Guru Hb added a comment -
            Changeset: 9bdcf1b4cc72
            Author: ghb
            Date: 2017-02-10 09:12 +0530
            URL: http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/9bdcf1b4cc72
            Show
            ghb Guru Hb added a comment - Changeset: 9bdcf1b4cc72 Author: ghb Date: 2017-02-10 09:12 +0530 URL: http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/9bdcf1b4cc72

              People

              • Assignee:
                ghb Guru Hb
                Reporter:
                shemnon Danno Ferrin (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                12 Start watching this issue

                Dates

                • Due:
                  Created:
                  Updated:
                  Resolved: