Adjust visibility and accessability checks.

Author: raphw

byte-buddy-dep/src/main/java/net/bytebuddy/description/method/MethodDescription.java

@@ -610,23 +610,21 @@ public int getActualModifiers(boolean manifest, Visibility visibility) {
          * {@inheritDoc}
          */
         public boolean isVisibleTo(TypeDescription typeDescription) {
-            return (isVirtual() || getDeclaringType().asErasure().isVisibleTo(typeDescription))
-                    && (isPublic()
-                    || typeDescription.equals(getDeclaringType().asErasure())
-                    || isProtected() && getDeclaringType().asErasure().isAssignableFrom(typeDescription)
-                    || !isPrivate() && typeDescription.isSamePackage(getDeclaringType().asErasure())
-                    || isPrivate() && typeDescription.isNestMateOf(getDeclaringType().asErasure()));
+            return getDeclaringType().asErasure().equals(typeDescription)
+                    || isPublic() && getDeclaringType().isPublic()
+                    || (isPublic() || isProtected()) && getDeclaringType().asErasure().isAssignableFrom(typeDescription)
+                    || !isPrivate() && getDeclaringType().asErasure().isSamePackage(typeDescription)
+                    || isPrivate() && getDeclaringType().asErasure().isNestMateOf(typeDescription);
         }
 
         /**
          * {@inheritDoc}
          */
         public boolean isAccessibleTo(TypeDescription typeDescription) {
-            return (isVirtual() || getDeclaringType().asErasure().isVisibleTo(typeDescription))
-                    && (isPublic()
-                    || typeDescription.equals(getDeclaringType().asErasure())
-                    || !isPrivate() && typeDescription.isSamePackage(getDeclaringType().asErasure()))
-                    || isPrivate() && typeDescription.isNestMateOf(getDeclaringType().asErasure());
+            return getDeclaringType().asErasure().equals(typeDescription)
+                    || isPublic() && getDeclaringType().isPublic()
+                    || !isPrivate() && getDeclaringType().asErasure().isSamePackage(typeDescription)
+                    || isPrivate() && getDeclaringType().asErasure().isNestMateOf(typeDescription);
         }
 
         /**

byte-buddy-dep/src/test/java/net/bytebuddy/description/method/AbstractMethodDescriptionTest.java

@@ -430,7 +430,7 @@ public void testMethodIsVisibleTo() throws Exception {
         assertThat(describe(PublicType.class.getDeclaredMethod("privateMethod"))
                 .isVisibleTo(TypeDescription.ForLoadedType.of(MethodDescriptionTestHelper.class)), is(false));
         assertThat(describe(PackagePrivateType.class.getDeclaredMethod("publicMethod"))
-                .isVisibleTo(TypeDescription.ForLoadedType.of(Object.class)), is(true));
+                .isVisibleTo(TypeDescription.ForLoadedType.of(Object.class)), is(false));
         assertThat(describe(PackagePrivateType.class.getDeclaredMethod("protectedMethod"))
                 .isVisibleTo(TypeDescription.ForLoadedType.of(Object.class)), is(false));
         assertThat(describe(PackagePrivateType.class.getDeclaredMethod("packagePrivateMethod"))
@@ -535,7 +535,7 @@ public void testMethodIsAccessibleTo() throws Exception {
         assertThat(describe(PublicType.class.getDeclaredMethod("privateMethod"))
                 .isAccessibleTo(TypeDescription.ForLoadedType.of(MethodDescriptionTestHelper.class)), is(false));
         assertThat(describe(PackagePrivateType.class.getDeclaredMethod("publicMethod"))
-                .isAccessibleTo(TypeDescription.ForLoadedType.of(Object.class)), is(true));
+                .isAccessibleTo(TypeDescription.ForLoadedType.of(Object.class)), is(false));
         assertThat(describe(PackagePrivateType.class.getDeclaredMethod("protectedMethod"))
                 .isAccessibleTo(TypeDescription.ForLoadedType.of(Object.class)), is(false));
         assertThat(describe(PackagePrivateType.class.getDeclaredMethod("packagePrivateMethod"))