Address PR feedback

packages/google_sign_in/google_sign_in_platform_interface/lib/google_sign_in_platform_interface.dart

@@ -71,7 +71,7 @@ abstract class GoogleSignInPlatform {
     throw UnimplementedError('init() has not been implemented.');
   }
 
-  /// Attempts to reuse pre-existing credentials to sign in again, without user interaction
+  /// Attempts to reuse pre-existing credentials to sign in again, without user interaction.
   Future<GoogleSignInUserData> signInSilently() async {
     throw UnimplementedError('signInSilently() has not been implemented.');
   }

packages/google_sign_in/google_sign_in_platform_interface/lib/src/method_channel_google_sign_in.dart

@@ -31,25 +31,26 @@ class MethodChannelGoogleSignIn extends GoogleSignInPlatform {
 
   @override
   Future<GoogleSignInUserData> signInSilently() {
-    return channel.invokeMapMethod<String, dynamic>('signInSilently').then(
-        (Map<String, dynamic> data) => nativeUserDataToPluginUserData(data));
+    return channel
+        .invokeMapMethod<String, dynamic>('signInSilently')
+        .then(getUserDataFromMap);
   }
 
   @override
   Future<GoogleSignInUserData> signIn() {
-    return channel.invokeMapMethod<String, dynamic>('signIn').then(
-        (Map<String, dynamic> data) => nativeUserDataToPluginUserData(data));
+    return channel
+        .invokeMapMethod<String, dynamic>('signIn')
+        .then(getUserDataFromMap);
   }
 
   @override
   Future<GoogleSignInTokenData> getTokens(
       {String email, bool shouldRecoverAuth = true}) {
-    return channel.invokeMapMethod<String, dynamic>(
-        'getTokens', <String, dynamic>{
+    return channel
+        .invokeMapMethod<String, dynamic>('getTokens', <String, dynamic>{
       'email': email,
       'shouldRecoverAuth': shouldRecoverAuth,
-    }).then(
-        (Map<String, dynamic> data) => nativeTokenDataToPluginTokenData(data));
+    }).then(getTokenDataFromMap);
   }
 
   @override

packages/google_sign_in/google_sign_in_platform_interface/lib/src/types.dart

@@ -19,11 +19,12 @@ class GoogleSignInUserData {
   bool operator ==(dynamic other) {
     if (identical(this, other)) return true;
     if (other is! GoogleSignInUserData) return false;
-    return other.displayName == displayName &&
-        other.email == email &&
-        other.id == id &&
-        other.photoUrl == photoUrl &&
-        other.idToken == idToken;
+    final GoogleSignInUserData otherUserData = other;
+    return otherUserData.displayName == displayName &&
+        otherUserData.email == email &&
+        otherUserData.id == id &&
+        otherUserData.photoUrl == photoUrl &&
+        otherUserData.idToken == idToken;
   }
 }
 
@@ -39,6 +40,8 @@ class GoogleSignInTokenData {
   bool operator ==(dynamic other) {
     if (identical(this, other)) return true;
     if (other is! GoogleSignInTokenData) return false;
-    return other.idToken == idToken && other.accessToken == accessToken;
+    final GoogleSignInTokenData otherTokenData = other;
+    return otherTokenData.idToken == idToken &&
+        otherTokenData.accessToken == accessToken;
   }
 }

packages/google_sign_in/google_sign_in_platform_interface/lib/src/utils.dart

@@ -1,6 +1,7 @@
 import '../google_sign_in_platform_interface.dart';
 
-GoogleSignInUserData nativeUserDataToPluginUserData(Map<String, dynamic> data) {
+/// Converts user data coming from native code into the proper platform interface type.
+GoogleSignInUserData getUserDataFromMap(Map<String, dynamic> data) {
   if (data == null) {
     return null;
   }
@@ -12,8 +13,8 @@ GoogleSignInUserData nativeUserDataToPluginUserData(Map<String, dynamic> data) {
       idToken: data['idToken']);
 }
 
-GoogleSignInTokenData nativeTokenDataToPluginTokenData(
-    Map<String, dynamic> data) {
+/// Converts token data coming from native code into the proper platform interface type.
+GoogleSignInTokenData getTokenDataFromMap(Map<String, dynamic> data) {
   if (data == null) {
     return null;
   }

packages/google_sign_in/google_sign_in_platform_interface/test/method_channel_google_sign_in_test.dart

@@ -30,9 +30,8 @@ const Map<String, dynamic> kDefaultResponses = <String, dynamic>{
   'getTokens': kTokenData,
 };
 
-final GoogleSignInUserData kUser = nativeUserDataToPluginUserData(kUserData);
-final GoogleSignInTokenData kToken =
-    nativeTokenDataToPluginTokenData(kTokenData);
+final GoogleSignInUserData kUser = getUserDataFromMap(kUserData);
+final GoogleSignInTokenData kToken = getTokenDataFromMap(kTokenData);
 
 void main() {
   TestWidgetsFlutterBinding.ensureInitialized();