From 3e353ea25e70e6adfa2d6b9b8f8cc0ca1a97be0f Mon Sep 17 00:00:00 2001
From: Brian Julian <bjj@google.com>
Date: Mon, 29 Jan 2024 16:36:38 +0000
Subject: Unhide AltitudeConverter method to perform non-blocking geoid height
 calculations.

Test: CtsLocationNoneTestCases
Bug: 304375846
Change-Id: I8db2db2a36a247c8e1f626d590a94c46a275c216
---
 .../java/android/location/altitude/AltitudeConverter.java    | 12 +++++++-----
 location/java/android/location/flags/location.aconfig        |  7 +++++++
 2 files changed, 14 insertions(+), 5 deletions(-)

(limited to 'location/java')

diff --git a/location/java/android/location/altitude/AltitudeConverter.java b/location/java/android/location/altitude/AltitudeConverter.java
index 6f8891216bed..461dafb91916 100644
--- a/location/java/android/location/altitude/AltitudeConverter.java
+++ b/location/java/android/location/altitude/AltitudeConverter.java
@@ -16,12 +16,14 @@
 
 package android.location.altitude;
 
+import android.annotation.FlaggedApi;
 import android.annotation.NonNull;
 import android.annotation.WorkerThread;
 import android.content.Context;
 import android.frameworks.location.altitude.GetGeoidHeightRequest;
 import android.frameworks.location.altitude.GetGeoidHeightResponse;
 import android.location.Location;
+import android.location.flags.Flags;
 
 import com.android.internal.location.altitude.GeoidMap;
 import com.android.internal.location.altitude.S2CellIdUtils;
@@ -213,12 +215,12 @@ public final class AltitudeConverter {
     }
 
     /**
-     * Same as {@link #addMslAltitudeToLocation(Context, Location)} except that data will not be
-     * loaded from raw assets. Returns true if a Mean Sea Level altitude is added to the
-     * {@code location}; otherwise, returns false and leaves the {@code location} unchanged.
-     *
-     * @hide
+     * Same as {@link #addMslAltitudeToLocation(Context, Location)} except that this method can be
+     * called on the main thread as data will not be loaded from raw assets. Returns true if a Mean
+     * Sea Level altitude is added to the {@code location}; otherwise, returns false and leaves the
+     * {@code location} unchanged.
      */
+    @FlaggedApi(Flags.FLAG_GEOID_HEIGHTS_VIA_ALTITUDE_HAL)
     public boolean addMslAltitudeToLocation(@NonNull Location location) {
         validate(location);
         MapParamsProto geoidHeightParams = GeoidMap.getGeoidHeightParams();
diff --git a/location/java/android/location/flags/location.aconfig b/location/java/android/location/flags/location.aconfig
index 32ad09c00d37..a96fe47f2381 100644
--- a/location/java/android/location/flags/location.aconfig
+++ b/location/java/android/location/flags/location.aconfig
@@ -7,6 +7,13 @@ flag {
     bug: "311210517"
 }
 
+flag {
+    name: "geoid_heights_via_altitude_hal"
+    namespace: "location"
+    description: "Flag for making geoid heights available via the Altitude HAL"
+    bug: "304375846"
+}
+
 flag {
     name: "gnss_api_navic_l1"
     namespace: "location"
-- 
cgit v1.2.3-59-g8ed1b