Remove redundant definesMember check.

Author: yilinwei

compiler/src/dotty/tools/dotc/typer/Namer.scala

@@ -908,7 +908,7 @@ class Namer { typer: Typer =>
           ||
           // remove synthetic constructor or method of a java Record if it clashes with a non-synthetic constructor
           (isJavaRecord(denot.owner)
-            && (denot.isConstructor || definesMember)
+            && denot.is(Method)
             && denot.owner.unforcedDecls.lookupAll(denot.name).exists(c => c != denot.symbol && c.info.matches(denot.info))
           )
         )

compiler/src/dotty/tools/dotc/typer/Typer.scala

@@ -2548,7 +2548,7 @@ class Typer(@constructorOnly nestingLevel: Int = 0) extends Namer
     def canBeInvalidated(sym: Symbol): Boolean =
       sym.is(Synthetic)
       && (desugar.isRetractableCaseClassMethodName(sym.name) ||
-         (sym.owner.is(JavaDefined) && sym.owner.derivesFrom(defn.JavaRecordClass)))
+         (sym.owner.is(JavaDefined) && sym.owner.derivesFrom(defn.JavaRecordClass) && sym.is(Method)))
 
     if !sym.info.exists then
       // it's a discarded method (synthetic case class method or synthetic java record constructor or overriden member), drop it