From 1959b63b61114def922e3302b38c347ee9d9b598 Mon Sep 17 00:00:00 2001 From: sakal Date: Tue, 7 Feb 2017 01:41:47 -0800 Subject: [PATCH] Remove Assert lint suppression. BUG=webrtc:6597 NOTRY=True Review-Url: https://codereview.webrtc.org/2668963002 Cr-Commit-Position: refs/heads/master@{#16460} --- tools-webrtc/android/suppressions.xml | 1 - .../sdk/android/api/org/webrtc/NetworkMonitorAutoDetect.java | 5 ++++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/tools-webrtc/android/suppressions.xml b/tools-webrtc/android/suppressions.xml index 6a36948c9f..52b0ad8371 100644 --- a/tools-webrtc/android/suppressions.xml +++ b/tools-webrtc/android/suppressions.xml @@ -6,7 +6,6 @@ - diff --git a/webrtc/sdk/android/api/org/webrtc/NetworkMonitorAutoDetect.java b/webrtc/sdk/android/api/org/webrtc/NetworkMonitorAutoDetect.java index 6bd727fd7f..c095485bbe 100644 --- a/webrtc/sdk/android/api/org/webrtc/NetworkMonitorAutoDetect.java +++ b/webrtc/sdk/android/api/org/webrtc/NetworkMonitorAutoDetect.java @@ -263,7 +263,10 @@ public class NetworkMonitorAutoDetect extends BroadcastReceiver { // ConnectivityManager.getDefaultNetwork() though this // may give confusing results with VPNs and is only // available with Android Marshmallow. - assert defaultNetId == INVALID_NET_ID; + if (defaultNetId != INVALID_NET_ID) { + throw new RuntimeException( + "Multiple connected networks of same type are not supported."); + } defaultNetId = networkToNetId(network); } }