Heartbeat logging passthrough for FIS for mobile platforms. (#853)

* Heartbeat logging passthrough for FIS for mobile platforms.

* Call CheckAndClearJniExceptions in LogHeartbeat

Author: AlmostMatt

installations/src/android/installations_android.cc

@@ -15,6 +15,7 @@
 #include "installations/src/android/installations_android.h"
 
 #include "app/src/util.h"
+#include "app/src/util_android.h"
 #include "installations/src/common.h"
 
 namespace firebase {
@@ -186,6 +187,19 @@ void InstallationsInternal::Cleanup() {
   }
 }
 
+void InstallationsInternal::LogHeartbeat(const firebase::App& app) {
+  // Calling the native getter is sufficient to cause a Heartbeat to be logged.
+  JNIEnv* env = app.GetJNIEnv();
+  jobject platform_app = app.GetPlatformApp();
+  jclass installations_class = installations::GetClass();
+  jobject installations_instance_local = env->CallStaticObjectMethod(
+      installations_class,
+      installations::GetMethodId(installations::kGetInstance), platform_app);
+  firebase::util::CheckAndClearJniExceptions(env);
+  env->DeleteLocalRef(installations_instance_local);
+  env->DeleteLocalRef(platform_app);
+}
+
 Future<std::string> InstallationsInternal::GetId() {
   const auto handle =
       future_impl_.SafeAlloc<std::string>(kInstallationsFnGetId);

installations/src/android/installations_android.h

@@ -34,6 +34,10 @@ class InstallationsInternal {
   explicit InstallationsInternal(const firebase::App& app);
   ~InstallationsInternal();
 
+  // Platform-specific method that causes a heartbeat to be logged.
+  // See go/firebase-platform-logging-design for more information.
+  static void LogHeartbeat(const firebase::App& app);
+
   Future<std::string> GetId();
   Future<std::string> GetIdLastResult();
 

installations/src/installations.cc

@@ -32,6 +32,9 @@ Installations* Installations::GetInstance(App* app) {
   // Return the Installations if it already exists.
   Installations* existing_installations = FindInstallations(app);
   if (existing_installations) {
+    // Log heartbeat data when instance getters are called.
+    // See go/firebase-platform-logging-design for more information.
+    InstallationsInternal::LogHeartbeat(*app);
     return existing_installations;
   }
 

installations/src/ios/installations_ios.h

@@ -42,6 +42,10 @@ class InstallationsInternal {
   explicit InstallationsInternal(const firebase::App& app);
   ~InstallationsInternal();
 
+  // Platform-specific method that causes a heartbeat to be logged.
+  // See go/firebase-platform-logging-design for more information.
+  static void LogHeartbeat(const firebase::App& app);
+
   Future<std::string> GetId();
   Future<std::string> GetIdLastResult();
 

installations/src/ios/installations_ios.mm

@@ -40,6 +40,12 @@
   // Destructor is necessary for ARC garbage collection.
 }
 
+void InstallationsInternal::LogHeartbeat(const firebase::App& app) {
+  // Calling the native getter is sufficient to cause a Heartbeat to be logged.
+  FIRApp *platform_app = app.GetPlatformApp();
+  [FIRInstallations installationsWithApp:platform_app];
+}
+
 bool InstallationsInternal::Initialized() const{
   return true;
 }

installations/src/stub/installations_stub.cc

@@ -26,6 +26,8 @@ InstallationsInternal::InstallationsInternal(const firebase::App& app)
 
 InstallationsInternal::~InstallationsInternal() {}
 
+void InstallationsInternal::LogHeartbeat(const firebase::App& app) {}
+
 bool InstallationsInternal::Initialized() const { return true; }
 
 void InstallationsInternal::Cleanup() {}

installations/src/stub/installations_stub.h

@@ -31,6 +31,10 @@ class InstallationsInternal {
   explicit InstallationsInternal(const firebase::App& app);
   ~InstallationsInternal();
 
+  // Platform-specific method that causes a heartbeat to be logged.
+  // See go/firebase-platform-logging-design for more information.
+  static void LogHeartbeat(const firebase::App& app);
+
   Future<std::string> GetId();
   Future<std::string> GetIdLastResult();