Refactor AuthInfo class to final AuthInfoUtil class with static methods

The AuthInfo class does not need to provide instance methods because the way it was used just to extract authentication info from request class so the AuthInfo constructor injection with HttpServletReqest does not work or necessary:
1. Change the name from AuthInfo to AuthInfoUtil to reflect that it is just a utility class with static methods.
2. Change all the public methods to become public static.
3. Update the unit test file.
4. Update the caller of this class to use the new AuthInfoUtil class.

[henry.saputra, 2010-11-24 17:30:46]

Hi,

Propose to refactor AuthInfo class to a final AuthInfoUtil class which provides
static utility methods instead of member instance methods.

The reason is because all the usage for AuthInfo currently just creating new
AuthInfo instance and call the method directly to get/set information from/to
request object so its not necessary to have instance public methods.

For example:
  new AuthInfo(req).setAuthType(handler.getName()).setSecurityToken(token);

Any review/suggestion is welcomed.

- Henry

[plindner1, 2010-11-28 00:19:29]

seems okay to me.

In general though both of these methods just feel wrong for some reason.  I'd
almost prefer to see us using guice-servlet @requestscoped annotations
instead...

[henry.saputra, 2010-11-28 06:49:35]

Committed as r1039836

[henry.saputra, 2010-11-28 06:50:35]

I can open a JIRA case if we want to move storing the auth info as
@requestscoped or as ThreadLocal.