commit 2654c8ed2b1b6dc025185eb5cf513e25a38c4595
author Nick Chalko <nchalko@google.com> Thu Feb 21 09:18:49 2019 -0800
committer Nick Chalko <nchalko@google.com> Wed Feb 27 22:17:59 2019 +0000
tree 3e5b2f0df929d32cc39f284af2294f128e716bd4
parent 5a9da78042abcd6cedec97c321042c4a74180a60

Turn on extcon event observing

selinux policy now includes sysfs_extcon
Submit after
https://android-review.googlesource.com/c/platform/system/sepolicy/+/907257
is merged

Bug: 124364409
Test: m services.core
Change-Id: I5401e36b5e752387cf7794dcca55da2270d46f66

services/core/java/com/android/server/ExtconUEventObserver.java

diff --git a/services/core/java/com/android/server/ExtconUEventObserver.java b/services/core/java/com/android/server/ExtconUEventObserver.java
index 775e4c8..6b42b3d 100644
--- a/services/core/java/com/android/server/ExtconUEventObserver.java
+++ b/services/core/java/com/android/server/ExtconUEventObserver.java
@@ -162,15 +162,6 @@
     /** Does the {@link /sys/class/extcon} directory exist */
     public static boolean extconExists() {
         File extconDir = new File("/sys/class/extcon");
-        boolean retVal = extconDir.exists() && extconDir.isDirectory();
-        // TODO(b/124364409): return the correct value after selinux policy is updated.
-        if (retVal) {
-            Slog.w(TAG, extconDir + " exists " + extconDir.exists() + " isDir "
-                    + extconDir.isDirectory()
-                    + " but reporting it does not exist until selinux policies are updated."
-                    + " see b/124364409"
-            );
-        }
-        return false;
+        return extconDir.exists() && extconDir.isDirectory();
     }
 }