RFR: 8012675: Javadoc for javax.swing.text.html.parser.Parser.handleComment() needs to be updated [v8]

[Damon Nguyen]

On Wed, 8 Jun 2022 17:01:23 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>>> the reasoning for the test originally using a temporary file to create HTML was to have an HTML file that could be opened itself and compared towards. I see that it's not necessary for this test, but was useful at the time. I thought it could still be useful, but this test with the HTML stored in a string is probably sufficient.
>> 
>> It could still be useful. An option to the test, say `-saveHTML`, could be used to get the HTML file to open in browser.
>> 
>> Yet I think it's easier to run the test on its own when you're developing it, no need to define the system property.
>
> At the same, I noticed a weird behaviour with the file. When I ran the test, class file was located in another directory but the HTML file was in the current directory. Calling `setPage` produced an exception. And `PassFailJFrame` started to behave erratically: it didn't close after clicking Pass or Fail. I didn't look into the reasons why. But it looks as bug in our manual testing framework…

I can look into this as well. Does this exception occur with the current version without the HTML file? I have no issues on my end.

-------------

PR: https://git.openjdk.java.net/jdk/pull/7446