[rfc][icedtea-web] additions to test-extensions needed for the new AWTFramework

Jiri Vanek jvanek at redhat.com
Mon Dec 17 08:42:20 PST 2012


On 12/13/2012 11:03 AM, Jana Fabrikova wrote:
> 2012-12-13  Jana Fabrikova  <jfabriko at redhat.com>
>
>      * /tests/test-extensions/net/sourceforge/jnlp/ServerAccess.java:
>      Added several new versions of method executeBrowser with lists
>      of ContentReaderListeners as arguments.
>
>      * /tests/test-extensions/net/sourceforge/jnlp/ProcessWrapper.java:
>      Added new versions of the constructor and methods for adding
>      ContentReaderListeners using List<ContentReaderListener>
>      instead of one ContentReaderListener.
>      Added a new version of constructor with less arguments (used
>      in case when several arguments would be passed as null, thus
>      causing ambiguity).
>
>      * /tests/reproducers/simple/SingeInstanceServiceTest/testcases/SingleInstanceTest.java:
>      Modified the call of executeBrowser method with null arguments
>      into a call of new method without the null arguments
>      (getting rid of an ambiguous call).
>
>
> I would like to ask for review of the attached patch,
> thank you,
> Jana

It looks much bettter. You forgot to fix the changelog (parentheses around affected and new methods. 
Listed with some luck all of them)
Still nasty comment:  //current browser may be
                                                             // null, but it does not
                                                             // matter

Should be on one line.

Although there are still some duplications I'm ok with them and after above changes ok for head.

J.



More information about the distro-pkg-dev mailing list