From 2de7d8ea38c0f2cd692dc0b0ada7e3b39d29ae5e Mon Sep 17 00:00:00 2001
From: Patrick Rohr <prohr@google.com>
Date: Tue, 26 Mar 2024 11:20:22 -0700
Subject: [PATCH] Remove extra handler#post inside setIncludeTestInterfaces

This was (mistakenly?) added in aosp/1267639 patchset 2; however, there
does not seem to be any apparent reason to post
trackAvailableInterfaces() to the back of the handler thread. All
operations that precede trackAvailableInterfaces() are synchronous.

Test: atest EthernetManagerTest
Change-Id: Ic8e888ce5d8909c4b7a00e26c54286c454ff3e75
---
 service-t/src/com/android/server/ethernet/EthernetTracker.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/service-t/src/com/android/server/ethernet/EthernetTracker.java b/service-t/src/com/android/server/ethernet/EthernetTracker.java
index bfcc171e8a..9c8fd99c39 100644
--- a/service-t/src/com/android/server/ethernet/EthernetTracker.java
+++ b/service-t/src/com/android/server/ethernet/EthernetTracker.java
@@ -460,7 +460,7 @@ public class EthernetTracker {
             if (!include) {
                 removeTestData();
             }
-            mHandler.post(() -> trackAvailableInterfaces());
+            trackAvailableInterfaces();
         });
     }