Hi Srini, The fix looks good, but what for is the arguments handling in the test?
Also, I would prefer to use our own base class (constructor must invoke overridable method, what if TreeItem is refactored in the future?) Hannes > Am 19.07.2017 um 10:45 schrieb Srinivas Dama <srinivas.d...@oracle.com>: > > Hi, > > Please review http://cr.openjdk.java.net/~sdama/8184241/webrev.00/ for > https://bugs.openjdk.java.net/browse/JDK-8184241. > > Regards, > Srinivas