use generics for AbstractFindByBuilder to avoid excessive casting

Author: asolntsev

java/src/org/openqa/selenium/support/AbstractFindByBuilder.java

@@ -22,9 +22,9 @@
 import java.util.Set;
 import org.openqa.selenium.By;
 
-public abstract class AbstractFindByBuilder {
+public abstract class AbstractFindByBuilder<T> {
 
-  public abstract By buildIt(Object annotation, Field field);
+  public abstract By buildIt(T annotation, Field field);
 
   protected By buildByFromFindBy(FindBy findBy) {
     assertValidFindBy(findBy);

java/src/org/openqa/selenium/support/FindAll.java

@@ -45,10 +45,9 @@
 public @interface FindAll {
   FindBy[] value();
 
-  class FindByBuilder extends AbstractFindByBuilder {
+  class FindByBuilder extends AbstractFindByBuilder<FindAll> {
     @Override
-    public By buildIt(Object annotation, Field field) {
-      FindAll findBys = (FindAll) annotation;
+    public By buildIt(FindAll findBys, Field field) {
       assertValidFindAll(findBys);
 
       FindBy[] findByArray = findBys.value();

java/src/org/openqa/selenium/support/FindBy.java

@@ -74,10 +74,9 @@
 
   String xpath() default "";
 
-  class FindByBuilder extends AbstractFindByBuilder {
+  class FindByBuilder extends AbstractFindByBuilder<FindBy> {
     @Override
-    public By buildIt(Object annotation, Field field) {
-      FindBy findBy = (FindBy) annotation;
+    public By buildIt(FindBy findBy, Field field) {
       assertValidFindBy(findBy);
 
       By ans = buildByFromShortFindBy(findBy);

java/src/org/openqa/selenium/support/FindBys.java

@@ -44,10 +44,9 @@
 public @interface FindBys {
   FindBy[] value();
 
-  class FindByBuilder extends AbstractFindByBuilder {
+  class FindByBuilder extends AbstractFindByBuilder<FindBys> {
     @Override
-    public By buildIt(Object annotation, Field field) {
-      FindBys findBys = (FindBys) annotation;
+    public By buildIt(FindBys findBys, Field field) {
       assertValidFindBys(findBys);
 
       FindBy[] findByArray = findBys.value();