From 3ad2c8d717a75f357b6e8f5de3ad42225438069a Mon Sep 17 00:00:00 2001 From: Jonas Oreland Date: Fri, 6 Sep 2024 14:13:54 +0200 Subject: [PATCH] Make getNumObservers @VisibleForTesting so that it can be tested outside of package org.webrtc Bug: webrtc:0 Change-Id: I66259bba8c724889df3ee22a8d6b9710a04ba671 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/361863 Reviewed-by: Jonas Oreland Reviewed-by: Harald Alvestrand Commit-Queue: Jonas Oreland Cr-Commit-Position: refs/heads/main@{#42973} --- sdk/android/api/org/webrtc/NetworkMonitor.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/sdk/android/api/org/webrtc/NetworkMonitor.java b/sdk/android/api/org/webrtc/NetworkMonitor.java index ca7e4d5656..428463c4bd 100644 --- a/sdk/android/api/org/webrtc/NetworkMonitor.java +++ b/sdk/android/api/org/webrtc/NetworkMonitor.java @@ -13,10 +13,12 @@ package org.webrtc; import android.content.Context; import android.os.Build; import androidx.annotation.Nullable; +import androidx.annotation.VisibleForTesting; import java.util.ArrayList; import java.util.List; import org.webrtc.NetworkChangeDetector; + /** * Borrowed from Chromium's * src/net/android/java/src/org/chromium/net/NetworkChangeNotifier.java @@ -349,7 +351,8 @@ public class NetworkMonitor { } // For testing only. - int getNumObservers() { + @VisibleForTesting + public int getNumObservers() { synchronized (networkChangeDetectorLock) { return numObservers; }