Added defensive EventExecutor to prevent potential threading issues.

PiperOrigin-RevId: 783821897

Author: Nicholas Ventimiglia

samples/HelloWorld/Assets/Scripts/GoogleMobileAdsConsentController.cs

@@ -154,37 +154,32 @@ void UpdatePrivacyButton()
         {
             if (_privacyButton != null)
             {
-                 MobileAdsEventExecutor.ExecuteInUpdate(() =>
-                {
-                    _privacyButton.interactable =
-                        ConsentInformation.PrivacyOptionsRequirementStatus ==
-                            PrivacyOptionsRequirementStatus.Required;
-
-                });
+                _privacyButton.interactable =
+                    ConsentInformation.PrivacyOptionsRequirementStatus ==
+                        PrivacyOptionsRequirementStatus.Required;
             }
         }
 
         void UpdateErrorPopup(string message)
         {
-            MobileAdsEventExecutor.ExecuteInUpdate(() =>
+            if (string.IsNullOrEmpty(message))
             {
-                if (string.IsNullOrEmpty(message))
-                {
-                    return;
-                }
-                if (_errorText != null)
-                {
-                    _errorText.text = message;
-                }
-                if (_errorPopup != null)
-                {
-                    _errorPopup.SetActive(true);
-                }
-                if (_privacyButton != null)
-                {
-                    _privacyButton.interactable = true;
-                }
-            });
+                return;
+            }
+
+            if (_errorText != null)
+            {
+                _errorText.text = message;
+            }
+
+            if (_errorPopup != null)
+            {
+                _errorPopup.SetActive(true);
+            }
+            if (_privacyButton != null)
+            {
+                _privacyButton.interactable = true;
+            }
         }
     }
 }

samples/HelloWorld/Assets/Scripts/GoogleMobileAdsController.cs

@@ -1,7 +1,6 @@
 using System;
 using System.Collections.Generic;
 using UnityEngine;
-using GoogleMobileAds.Common;
 using GoogleMobileAds.Api;
 using GoogleMobileAds.Ump.Api;
 
@@ -42,6 +41,11 @@ private void Start()
             // This setting makes iOS behave consistently with Android.
             MobileAds.SetiOSAppPauseOnBackground(true);
 
+            // When true all events raised by GoogleMobileAds will be raised
+            // on the Unity main thread. The default value is false.
+            // https://developers.google.com/admob/unity/quick-start#raise_ad_events_on_the_unity_main_thread
+            MobileAds.RaiseAdEventsOnUnityMainThread = true;
+
             // Configure your RequestConfiguration with Child Directed Treatment
             // and the Test Device Ids.
             MobileAds.SetRequestConfiguration(new RequestConfiguration
@@ -101,6 +105,7 @@ private void InitializeGoogleMobileAds()
 
             // Initialize the Google Mobile Ads Unity plugin.
             Debug.Log("Google Mobile Ads Initializing.");
+
             // [START initialize_sdk]
             MobileAds.Initialize((InitializationStatus initstatus) =>
             {
@@ -126,11 +131,6 @@ private void InitializeGoogleMobileAds()
 
                 Debug.Log("Google Mobile Ads initialization complete.");
                 _isInitialized = true;
-
-                // Google Mobile Ads events are raised off the Unity Main thread. If you need to
-                // access UnityEngine objects after initialization,
-                // use MobileAdsEventExecutor.ExecuteInUpdate(). For more information, see:
-                // https://developers.google.com/admob/unity/global-settings#raise_ad_events_on_the_unity_main_thread
             });
             // [END initialize_sdk]
         }