Skip to content

Commit 9d4553a

Browse files
authored
Enable explicit api for AQS (#7375)
1 parent bb1c751 commit 9d4553a

File tree

4 files changed

+13
-11
lines changed

4 files changed

+13
-11
lines changed

firebase-sessions/firebase-sessions.gradle.kts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,8 @@ android {
6060
}
6161
}
6262

63+
kotlin { explicitApi() }
64+
6365
tasks.withType(org.jetbrains.kotlin.gradle.tasks.KaptGenerateStubs::class.java).configureEach {
6466
kotlinOptions.jvmTarget = "1.8"
6567
}

firebase-sessions/src/main/kotlin/com/google/firebase/sessions/api/CrashEventReceiver.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ import com.google.firebase.sessions.SharedSessionRepository
2525
* This object provides a static-like entry point that Crashlytics calls to inform Sessions a fatal
2626
* crash has occurred.
2727
*/
28-
object CrashEventReceiver {
28+
public object CrashEventReceiver {
2929
@VisibleForTesting internal lateinit var sharedSessionRepository: SharedSessionRepository
3030

3131
/**
@@ -38,7 +38,7 @@ object CrashEventReceiver {
3838
* @see SharedSessionRepository.appBackground
3939
*/
4040
@JvmStatic
41-
fun notifyCrashOccurred() {
41+
public fun notifyCrashOccurred() {
4242
try {
4343
if (!::sharedSessionRepository.isInitialized) {
4444
sharedSessionRepository = SharedSessionRepository.instance

firebase-sessions/src/main/kotlin/com/google/firebase/sessions/api/FirebaseSessionsDependencies.kt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,15 +30,15 @@ import kotlinx.coroutines.sync.withLock
3030
*
3131
* This is important because the Sessions SDK starts up before dependent SDKs.
3232
*/
33-
object FirebaseSessionsDependencies {
33+
public object FirebaseSessionsDependencies {
3434
private val dependencies = synchronizedMap(mutableMapOf<SessionSubscriber.Name, Dependency>())
3535

3636
/**
3737
* Add a subscriber as a dependency to the Sessions SDK. Every dependency must register itself, or
3838
* the Sessions SDK will never generate a session.
3939
*/
4040
@JvmStatic
41-
fun addDependency(subscriberName: SessionSubscriber.Name) {
41+
public fun addDependency(subscriberName: SessionSubscriber.Name) {
4242
if (dependencies.containsKey(subscriberName)) {
4343
Log.d(TAG, "Dependency $subscriberName already added.")
4444
return
@@ -54,7 +54,7 @@ object FirebaseSessionsDependencies {
5454
* return.
5555
*/
5656
@JvmStatic
57-
fun register(subscriber: SessionSubscriber) {
57+
public fun register(subscriber: SessionSubscriber) {
5858
val subscriberName = subscriber.sessionSubscriberName
5959
val dependency = getDependency(subscriberName)
6060

firebase-sessions/src/main/kotlin/com/google/firebase/sessions/api/SessionSubscriber.kt

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -19,20 +19,20 @@ package com.google.firebase.sessions.api
1919
import androidx.annotation.Discouraged
2020

2121
/** [SessionSubscriber] is an interface that dependent SDKs must implement. */
22-
interface SessionSubscriber {
22+
public interface SessionSubscriber {
2323
/** [SessionSubscriber.Name]s are used for identifying subscribers. */
24-
enum class Name {
24+
public enum class Name {
2525
CRASHLYTICS,
2626
PERFORMANCE,
2727
@Discouraged(message = "This is for testing purposes only.") MATT_SAYS_HI,
2828
}
2929

3030
/** [SessionDetails] contains session data passed to subscribers whenever the session changes */
31-
data class SessionDetails(val sessionId: String)
31+
public data class SessionDetails(val sessionId: String)
3232

33-
fun onSessionChanged(sessionDetails: SessionDetails)
33+
public fun onSessionChanged(sessionDetails: SessionDetails)
3434

35-
val isDataCollectionEnabled: Boolean
35+
public val isDataCollectionEnabled: Boolean
3636

37-
val sessionSubscriberName: Name
37+
public val sessionSubscriberName: Name
3838
}

0 commit comments

Comments
 (0)