commit b6f4d96f42e989edb5356664fc3740544d33be2f
author Mark Tabry <mtab@google.com> Fri Dec 13 13:02:23 2019 -0800
committer Mark Tabry <mtab@google.com> Fri Dec 13 22:04:42 2019 +0000
tree 897706efda8055540a6b2627be61fb0f67357c12
parent ff78c7c33e18927e47bf2aec48eceef37fd8f6b7

Remove exception throwing for sequence number wraparound.

Sequence number wraparounds will be handled client side. Throwing an
exception here only affects the setLayerOffering Binder calls, not the
Car service, leaving VMS in a broken state.

Bug: 79604831
Test: atest AndroidCarApiTest CarServiceTest CarServiceUnitTest
Change-Id: I63d5b77a80c282b12f11979f443bbd008f0474fe

service/src/com/android/car/VmsLayersAvailability.java

diff --git a/service/src/com/android/car/VmsLayersAvailability.java b/service/src/com/android/car/VmsLayersAvailability.java
index 2e17a89..a9bbc7a 100644
--- a/service/src/com/android/car/VmsLayersAvailability.java
+++ b/service/src/com/android/car/VmsLayersAvailability.java
@@ -109,9 +109,6 @@
             mPotentialLayersAndPublishers.clear();
             mAvailableAssociatedLayers = Collections.EMPTY_SET;
             mUnavailableAssociatedLayers = Collections.EMPTY_SET;
-            if (mSeq + 1 < mSeq) {
-                throw new IllegalStateException("Sequence is about to loop");
-            }
             mSeq += 1;
         }
     }