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

Hannes Wallnöfer hannes.wallnoefer at oracle.com
Fri Jan 17 12:00:58 UTC 2020


Looks good. Thanks for the manual testing/checking.

Hannes

> Am 16.01.2020 um 22:35 schrieb Pavel Rappo <pavel.rappo at oracle.com>:
> 
> 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