Thanks for catching; no, those lines should go; sorry, I should have
caught them in self-review.
-- Jon
On 3/19/20 5:18 AM, Pavel Rappo wrote:
These lines are commented out:
65 // /**
66 // * The HTML tree for main tag.
67 // */
68 // private final HtmlTree mainTree = HtmlTree.MAIN();
69
Do you intend to leave them like this?
Otherwise, looks good to me. Thanks.
-Pavel
On 19 Mar 2020, at 00:43, Jonathan Gibbons <[email protected]> wrote:
Please review a simple localized fix to construct the System Properties page
using the recently-new standard flex-box layout.
For reference, the modified code was based on equivalent code in
SingleIndexWriter.generateIndexFile.
The existing test for the page is extended with additional checks for the new
behavior.
-- Jon
JBS: https://bugs.openjdk.java.net/browse/JDK-8240476
Webrev: http://cr.openjdk.java.net/~jjg/8240476/webrev.00/index.html
