ClassUtils.getMostSpecificMethod uses Class.getMethod code path in case of a public method

This should be significantly faster than our standard algorithm, for a very common case. Motivated by SPR-9802, even if the fix there uses a different approach, with transaction name determination not calling getMostSpecificMethod at all anymore.

Issue: SPR-9802

Author: jhoeller

spring-core/src/main/java/org/springframework/util/ClassUtils.java

@@ -724,7 +724,7 @@ public static boolean hasAtLeastOneMethodWithName(Class<?> clazz, String methodN
 	 * Call {@link org.springframework.core.BridgeMethodResolver#findBridgedMethod}
 	 * if bridge method resolution is desirable (e.g. for obtaining metadata from
 	 * the original method definition).
-	 * <p><b>NOTE:</b>Since Spring 3.1.1, if java security settings disallow reflective
+	 * <p><b>NOTE:</b> Since Spring 3.1.1, if Java security settings disallow reflective
 	 * access (e.g. calls to {@code Class#getDeclaredMethods} etc, this implementation
 	 * will fall back to returning the originally provided method.
 	 * @param method the method to be invoked, which may come from an interface
@@ -734,17 +734,28 @@ public static boolean hasAtLeastOneMethodWithName(Class<?> clazz, String methodN
 	 * <code>targetClass</code> doesn't implement it or is <code>null</code>
 	 */
 	public static Method getMostSpecificMethod(Method method, Class<?> targetClass) {
-		Method specificMethod = null;
 		if (method != null && isOverridable(method, targetClass) &&
 				targetClass != null && !targetClass.equals(method.getDeclaringClass())) {
 			try {
-				specificMethod = ReflectionUtils.findMethod(targetClass, method.getName(), method.getParameterTypes());
-			} catch (AccessControlException ex) {
-				// security settings are disallowing reflective access; leave
-				// 'specificMethod' null and fall back to 'method' below
+				if (Modifier.isPublic(method.getModifiers())) {
+					try {
+						return targetClass.getMethod(method.getName(), method.getParameterTypes());
+					}
+					catch (NoSuchMethodException ex) {
+						return method;
+					}
+				}
+				else {
+					Method specificMethod =
+							ReflectionUtils.findMethod(targetClass, method.getName(), method.getParameterTypes());
+					return (specificMethod != null ? specificMethod : method);
+				}
+			}
+			catch (AccessControlException ex) {
+				// Security settings are disallowing reflective access; fall back to 'method' below.
 			}
 		}
-		return (specificMethod != null ? specificMethod : method);
+		return method;
 	}
 
 	/**
@@ -1150,5 +1161,4 @@ public static boolean isCglibProxyClassName(String className) {
 		return (className != null && className.contains(CGLIB_CLASS_SEPARATOR));
 	}
 
-
 }