Fix analyze errors

Author: peixinli

packages/google_sign_in/google_sign_in_web/lib/src/js_interop/gapi.dart

@@ -10,15 +10,15 @@
 
 // https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/gapi
 
-// ignore_for_file: public_member_api_docs, unused_element, sort_constructors_first, prefer_generic_function_type_aliases
+// ignore_for_file: public_member_api_docs,
 
 @JS()
 library gapi;
 
 import 'package:js/js.dart';
 
 // Module gapi
-typedef void LoadCallback(
+typedef LoadCallback = void Function(
     [dynamic args1,
     dynamic args2,
     dynamic args3,
@@ -28,6 +28,11 @@ typedef void LoadCallback(
 @anonymous
 @JS()
 abstract class LoadConfig {
+  external factory LoadConfig(
+      {LoadCallback callback,
+        Function? onerror,
+        num? timeout,
+        Function? ontimeout});
   external LoadCallback get callback;
   external set callback(LoadCallback v);
   external Function? get onerror;
@@ -36,11 +41,6 @@ abstract class LoadConfig {
   external set timeout(num? v);
   external Function? get ontimeout;
   external set ontimeout(Function? v);
-  external factory LoadConfig(
-      {LoadCallback callback,
-      Function? onerror,
-      num? timeout,
-      Function? ontimeout});
 }
 
 /*type CallbackOrConfig = LoadConfig | LoadCallback;*/

packages/google_sign_in/google_sign_in_web/lib/src/js_interop/gapiauth2.dart

@@ -12,7 +12,9 @@
 
 // https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/gapi.auth2
 
-// ignore_for_file: public_member_api_docs, unused_element, non_constant_identifier_names, sort_constructors_first, always_specify_types, strict_raw_type
+// ignore_for_file: public_member_api_docs, non_constant_identifier_names,
+// * non_constant_identifier_names required to be able to use the same parameter
+//   names as the underlying library.
 
 @JS()
 library gapiauth2;
@@ -122,6 +124,15 @@ abstract class CurrentUser {
 @anonymous
 @JS()
 abstract class SigninOptions {
+  external factory SigninOptions(
+      {String app_package_name,
+        bool fetch_basic_profile,
+        String prompt,
+        String scope,
+        String /*'popup'|'redirect'*/ ux_mode,
+        String redirect_uri,
+        String login_hint});
+
   /// The package name of the Android app to install over the air.
   /// See Android app installs from your web site:
   /// https://developers.google.com/identity/sign-in/web/android-app-installs
@@ -162,15 +173,6 @@ abstract class SigninOptions {
   // https://developers.google.com/identity/protocols/OpenIDConnect?hl=en#authenticationuriparameters
   external String? get login_hint;
   external set login_hint(String? v);
-
-  external factory SigninOptions(
-      {String app_package_name,
-      bool fetch_basic_profile,
-      String prompt,
-      String scope,
-      String /*'popup'|'redirect'*/ ux_mode,
-      String redirect_uri,
-      String login_hint});
 }
 
 /// Definitions by: John <https://github.com/jhcao23>
@@ -179,23 +181,35 @@ abstract class SigninOptions {
 @anonymous
 @JS()
 abstract class OfflineAccessOptions {
+  external factory OfflineAccessOptions(
+      {String scope,
+        String /*'select_account'|'consent'*/ prompt,
+        String app_package_name});
   external String? get scope;
   external set scope(String? v);
   external String? /*'select_account'|'consent'*/ get prompt;
   external set prompt(String? /*'select_account'|'consent'*/ v);
   external String? get app_package_name;
   external set app_package_name(String? v);
-  external factory OfflineAccessOptions(
-      {String scope,
-      String /*'select_account'|'consent'*/ prompt,
-      String app_package_name});
 }
 
 /// Interface that represents the different configuration parameters for the gapi.auth2.init method.
 /// Reference: https://developers.google.com/api-client-library/javascript/reference/referencedocs#gapiauth2clientconfig
 @anonymous
 @JS()
 abstract class ClientConfig {
+  external factory ClientConfig({
+    String client_id,
+    String cookie_policy,
+    String scope,
+    bool fetch_basic_profile,
+    String? hosted_domain,
+    String openid_realm,
+    String /*'popup'|'redirect'*/ ux_mode,
+    String redirect_uri,
+    String plugin_name,
+  });
+
   /// The app's client ID, found and created in the Google Developers Console.
   external String? get client_id;
   external set client_id(String? v);
@@ -238,18 +252,6 @@ abstract class ClientConfig {
   /// See: https://github.com/flutter/flutter/issues/88084
   external String? get plugin_name;
   external set plugin_name(String? v);
-
-  external factory ClientConfig({
-    String client_id,
-    String cookie_policy,
-    String scope,
-    bool fetch_basic_profile,
-    String? hosted_domain,
-    String openid_realm,
-    String /*'popup'|'redirect'*/ ux_mode,
-    String redirect_uri,
-    String plugin_name,
-  });
 }
 
 @JS('gapi.auth2.SigninOptionsBuilder')
@@ -276,6 +278,7 @@ abstract class BasicProfile {
 @anonymous
 @JS()
 abstract class AuthResponse {
+
   external String? get access_token;
   external set access_token(String? v);
   external String? get id_token;
@@ -290,20 +293,23 @@ abstract class AuthResponse {
   external set first_issued_at(num? v);
   external num? get expires_at;
   external set expires_at(num? v);
-  external factory AuthResponse(
-      {String? access_token,
-      String? id_token,
-      String? login_hint,
-      String? scope,
-      num? expires_in,
-      num? first_issued_at,
-      num? expires_at});
 }
 
 /// Reference: https://developers.google.com/api-client-library/javascript/reference/referencedocs#gapiauth2authorizeconfig
 @anonymous
 @JS()
 abstract class AuthorizeConfig {
+  external factory AuthorizeConfig(
+      {String client_id,
+        String scope,
+        String response_type,
+        String prompt,
+        String cookie_policy,
+        String hosted_domain,
+        String login_hint,
+        String app_package_name,
+        String openid_realm,
+        bool include_granted_scopes});
   external String get client_id;
   external set client_id(String v);
   external String get scope;
@@ -324,23 +330,22 @@ abstract class AuthorizeConfig {
   external set openid_realm(String? v);
   external bool? get include_granted_scopes;
   external set include_granted_scopes(bool? v);
-  external factory AuthorizeConfig(
-      {String client_id,
-      String scope,
-      String response_type,
-      String prompt,
-      String cookie_policy,
-      String hosted_domain,
-      String login_hint,
-      String app_package_name,
-      String openid_realm,
-      bool include_granted_scopes});
 }
 
 /// Reference: https://developers.google.com/api-client-library/javascript/reference/referencedocs#gapiauth2authorizeresponse
 @anonymous
 @JS()
 abstract class AuthorizeResponse {
+  external factory AuthorizeResponse(
+      {String access_token,
+        String id_token,
+        String code,
+        String scope,
+        num expires_in,
+        num first_issued_at,
+        num expires_at,
+        String error,
+        String error_subtype});
   external String get access_token;
   external set access_token(String v);
   external String get id_token;
@@ -359,16 +364,6 @@ abstract class AuthorizeResponse {
   external set error(String v);
   external String get error_subtype;
   external set error_subtype(String v);
-  external factory AuthorizeResponse(
-      {String access_token,
-      String id_token,
-      String code,
-      String scope,
-      num expires_in,
-      num first_issued_at,
-      num expires_at,
-      String error,
-      String error_subtype});
 }
 
 /// A GoogleUser object represents one user account.
@@ -498,6 +493,4 @@ external void render(
 abstract class Promise<T> {
   external factory Promise(
       void Function(void Function(T result) resolve, Function reject) executor);
-  external Promise then(void Function(T result) onFulfilled,
-      [Function onRejected]);
 }