Skip to content

Commit f9c3bca

Browse files
committed
Merge branch 'master' of https://github.com/firebase/firebase-android-sdk into rpf-perf-ktx-trace
2 parents acf9a70 + f89b09b commit f9c3bca

File tree

4 files changed

+11
-10
lines changed

4 files changed

+11
-10
lines changed

appcheck/firebase-appcheck-safetynet/src/main/java/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,10 @@
2323
/**
2424
* Implementation of an {@link AppCheckProviderFactory} that builds {@link
2525
* SafetyNetAppCheckProvider}s. This is the default implementation.
26+
*
27+
* @deprecated Use {@code PlayIntegrityAppCheckProviderFactory} instead.
2628
*/
29+
@Deprecated
2730
public class SafetyNetAppCheckProviderFactory implements AppCheckProviderFactory {
2831

2932
private static final SafetyNetAppCheckProviderFactory instance =
@@ -34,7 +37,10 @@ private SafetyNetAppCheckProviderFactory() {}
3437
/**
3538
* Gets an instance of this class for installation into a {@link
3639
* com.google.firebase.appcheck.FirebaseAppCheck} instance.
40+
*
41+
* @deprecated Use {@code PlayIntegrityAppCheckProviderFactory#getInstance} instead.
3742
*/
43+
@Deprecated
3844
@NonNull
3945
public static SafetyNetAppCheckProviderFactory getInstance() {
4046
return instance;

firebase-config/src/main/java/com/google/firebase/remoteconfig/internal/ConfigFetchHttpClient.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -413,7 +413,7 @@ private static ConfigContainer extractConfigs(JSONObject fetchResponse, Date fet
413413
// Do nothing if entries do not exist.
414414
}
415415
if (entries != null) {
416-
containerBuilder.replaceConfigsWith(entries);
416+
containerBuilder = containerBuilder.replaceConfigsWith(entries);
417417
}
418418

419419
JSONArray experimentDescriptions = null;
@@ -423,7 +423,7 @@ private static ConfigContainer extractConfigs(JSONObject fetchResponse, Date fet
423423
// Do nothing if entries do not exist.
424424
}
425425
if (experimentDescriptions != null) {
426-
containerBuilder.withAbtExperiments(experimentDescriptions);
426+
containerBuilder = containerBuilder.withAbtExperiments(experimentDescriptions);
427427
}
428428

429429
JSONObject personalizationMetadata = null;
@@ -433,7 +433,7 @@ private static ConfigContainer extractConfigs(JSONObject fetchResponse, Date fet
433433
// Do nothing if personalizationMetadata does not exist.
434434
}
435435
if (personalizationMetadata != null) {
436-
containerBuilder.withPersonalizationMetadata(personalizationMetadata);
436+
containerBuilder = containerBuilder.withPersonalizationMetadata(personalizationMetadata);
437437
}
438438

439439
return containerBuilder.build();

firebase-config/src/test/java/com/google/firebase/remoteconfig/internal/ConfigFetchHandlerTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1033,10 +1033,10 @@ private static ConfigContainer extractConfigs(JSONObject fetchResponse, Date fet
10331033
ConfigContainer.newBuilder().withFetchTime(fetchTime);
10341034

10351035
JSONObject entries = fetchResponse.getJSONObject(ENTRIES);
1036-
containerBuilder.replaceConfigsWith(entries);
1036+
containerBuilder = containerBuilder.replaceConfigsWith(entries);
10371037

10381038
JSONArray experimentDescriptions = fetchResponse.getJSONArray(EXPERIMENT_DESCRIPTIONS);
1039-
containerBuilder.withAbtExperiments(experimentDescriptions);
1039+
containerBuilder = containerBuilder.withAbtExperiments(experimentDescriptions);
10401040

10411041
return containerBuilder.build();
10421042
}

firebase-perf/CHANGELOG.md

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,5 @@
11
# Unreleased
22

3-
* [fixed] Fixed a bug where screen traces were not capturing frame metrics for multi-activity apps.
4-
* [feature] Added support for measuring screen performance metrics for "Activity Fragments" out-of-the-box.
5-
* [fixed] Excluded custom attributes whose key/value length is 0.
6-
7-
83
# 20.2.0
94
* [unchanged] Updated to accommodate the release of the updated
105
[perfmon] Kotlin extensions library.

0 commit comments

Comments
 (0)