Removes SpringJunit4ClassRunner from SanityCheckIT in fcrepo-webapp since we're
relying on jetty-maven-plugin to start/stop the container for testing.
Uses build-helper-maven-plugin:reserve-network-ports to set test.port property
for all integration tests. The property ${test.port} is made available to the tests
via maven-failsafe-plugin.
Cleans up imports. Switches from junit.framework.Assert.assertEquals (which is deprecated)
to org.junit.Assert.assertEquals.
minor: cleans up imports, formatting
Removed dependency on HTTP APIs for testing per: https://www.pivotaltracker.com/story/show/47962441
Removed unused import
Closes the InputStream after we're done with it.
Demonstrated try-with-resources
Adds null checks for apply(F), throwing an NPE if parameter is null as evidently
required by the @Nullable contract of apply() and as FindBug's
NP_PARAMETER_MUST_BE_NONNULL_BUT_MARKED_AS_NULLABLE
has gently reminded.
bump lock timeout back up, seems to make matters worse
Another attempt to resolve FindBug's
NP_PARAMETER_MUST_BE_NONNULL_BUT_MARKED_AS_NULLABLE
warning.

Also see: http://code.google.com/p/guava-libraries/issues/detail?id=920
Corrects some checkstyle, pmd issues with lack of braces on if statements and
unnecessary use of toString() on Strings
Cleans up some more FindBugs warnings
minor: tweaks to test clearing PMD warnings
bump to ISPN 5.3.0.Alpha1
remove synchronous updateRepositroySize code; calculating the repository size is now done by querying the JCR index