RFR [15] 8236700: Upgrading JSZip from v3.1.5 to v3.2.2

Pavel Rappo pavel.rappo at oracle.com
Thu Jan 16 21:35:42 UTC 2020


Hello,

Please review the following change for https://bugs.openjdk.java.net/browse/JDK-8236700:

    http://cr.openjdk.java.net/~prappo/8236700/webrev.00/

This change upgrades the JSZip JavaScript library, used in Javadoc Search [1],
to v3.22. I tested this change as follows.

1. Commented out lines 90 through 104 in
src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/resources/script.js:

    // if (!moduleSearchIndex) {
    //     createElem(doc, tag, 'module-search-index.js');
    // }
    // if (!packageSearchIndex) {
    //     createElem(doc, tag, 'package-search-index.js');
    // }
    // if (!typeSearchIndex) {
    //     createElem(doc, tag, 'type-search-index.js');
    // }
    // if (!memberSearchIndex) {
    //     createElem(doc, tag, 'member-search-index.js');
    // }
    // if (!tagSearchIndex) {
    //     createElem(doc, tag, 'tag-search-index.js');
    // }

This effectively makes *-search-index.js, the 5 non-zipped index files above,
unavailable. So the search UI is forced to use the zipped files and, therefore,
the JSZip library.
2. Built docs using `make docs`
3. Started a simple HTTP server in build/.../images/docs as there needs to be a
web server to serve the zipped files, via HTTP requests, to the search UI.
4. Cleared the cache of the browser
5. Opened the main docs page, index.html, in the browser and verified that only
the *-search-index.zip files were transferred.
6. Checked that the search UI produced the expected results for typical search
terms such as list, thread, string, stream, etc.
7. Checked the same for a module, package, member, and tag search terms.
This is to make sure that each of the 5 index files are accessed.

Thanks,
-Pavel

--------------------------------------------------------------------------------
[1] https://bugs.openjdk.java.net/browse/JDK-8044243



More information about the javadoc-dev mailing list