oauth issue 61: Java 7 implementation of FileCredentialStore

oauth issue 61: Java 7 implementation of FileCredentialStore

[ngmiceli, 2012-12-10 19:18:34]

LGTM with one thought

https://codereview.appspot.com/6904070/diff/1/google-oauth-client-java7/src/m...
File
google-oauth-client-java7/src/main/java/com/google/api/client/extensions/java7/auth/oauth2/FileCredentialStore2.java
(right):

https://codereview.appspot.com/6904070/diff/1/google-oauth-client-java7/src/m...
google-oauth-client-java7/src/main/java/com/google/api/client/extensions/java7/auth/oauth2/FileCredentialStore2.java:31:
public class FileCredentialStore2 extends FileCredentialStore {
[optional] I always disliked "XXX2" as a class name because its not at all
intuitive what the difference is without having to go and check the Javadoc. 
Unfortunately, its hard to think of a good alternative. FileCredentialStoreJava7
is the only other thing that comes to my mind.

[yanivi, 2012-12-10 21:12:33]

Rename done.  Also added a unit test, including a Java 7 only test for symbolic
link.

[ngmiceli, 2012-12-10 21:38:31]

On 2012/12/10 21:12:33, yanivi wrote:
> Rename done.  Also added a unit test, including a Java 7 only test for
symbolic
> link.

LGTM